Re: Review Request 27829: ACCUMULO-1085

2014-11-24 Thread Eric Newton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27829/ --- (Updated Nov. 24, 2014, 1:53 p.m.) Review request for accumulo. Changes

Re: Review Request 27829: ACCUMULO-1085

2014-11-24 Thread Eric Newton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27829/ --- (Updated Nov. 24, 2014, 8:15 p.m.) Review request for accumulo. Bugs:

Re: Review Request 27829: ACCUMULO-1085

2014-11-24 Thread Eric Newton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27829/ --- (Updated Nov. 24, 2014, 8:33 p.m.) Review request for accumulo. Bugs:

Review Request 27829: ACCUMULO-1085

2014-11-10 Thread Eric Newton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27829/ --- Review request for accumulo. Repository: accumulo Description --- Allow

Re: Review Request 27829: ACCUMULO-1085

2014-11-10 Thread Mike Drob
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27829/#review60681 --- core/src/main/java/org/apache/accumulo/core/conf/Property.java

Re: Review Request 27829: ACCUMULO-1085

2014-11-10 Thread Eric Newton
On Nov. 10, 2014, 9:38 p.m., Mike Drob wrote: test/src/test/java/org/apache/accumulo/test/AssignmentThreadsIT.java, line 34 https://reviews.apache.org/r/27829/diff/1/?file=757270#file757270line34 Plkease add a test that only one recovery will happen at a time. Any idea how I can

Re: Review Request 27829: ACCUMULO-1085

2014-11-10 Thread Josh Elser
On Nov. 10, 2014, 9:38 p.m., Mike Drob wrote: test/src/test/java/org/apache/accumulo/test/AssignmentThreadsIT.java, line 34 https://reviews.apache.org/r/27829/diff/1/?file=757270#file757270line34 Plkease add a test that only one recovery will happen at a time. Eric Newton

Re: Review Request 27829: ACCUMULO-1085

2014-11-10 Thread Josh Elser
On Nov. 10, 2014, 9:38 p.m., Mike Drob wrote: test/src/test/java/org/apache/accumulo/test/AssignmentThreadsIT.java, line 44 https://reviews.apache.org/r/27829/diff/1/?file=757270#file757270line44 Is this just chars 0-9 a-f? Eric Newton wrote: yes; originally I left the