Home | About | Sematext search-lucene.com search-hadoop.com
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB
 Search Hadoop and all its subprojects:

Switch to Threaded View
Accumulo >> mail # dev >> ReviewBoard


3 things:

#1
ReviewBoard has been really buggy lately. Has anybody else noticed this?

Examples: errors when publishing a review, but review still gets
published, or moving to a second page on a diff, during a review, but
getting an error and losing your unpublished comments from the
previous page.

#2
I've noticed that ReviewBoard is being used more frequently lately,
and for tiny, relatively trivial patches. ReviewBoard is great (when
it's working), but I'm not sure it is strictly necessary to submit all
contributed patches to ReviewBoard. It's fine if you wish to use it (I
don't want to discourage it)... but I just don't want contributors
getting the impression that it's a requirement.

#3
Don't forget to close out your reviews after the patch has been applied.

--
Christopher L Tubbs II
http://gravatar.com/ctubbsii
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB