Re: Review Request 18444: ACCUMULO-2399 Continuous* wait a bit for scanners

2014-02-25 Thread Bill Havanki
On Feb. 25, 2014, 9:01 a.m., Bill Havanki wrote: src/server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousUtil.java, line 46 https://reviews.apache.org/r/18444/diff/1/?file=502880#file502880line46 What if, due to the cluster environment, it always takes

Re: Review Request 18444: ACCUMULO-2399 Continuous* wait a bit for scanners

2014-02-25 Thread Mike Drob
On Feb. 25, 2014, 12:23 a.m., Sean Busbey wrote: Is there a disadvantage to instead updating the README to require creating the table ahead of time? That way we could avoid this code, and we could remove the table creation code from the writers. As a side effect, there'd also be a

Re: Review Request 18444: ACCUMULO-2399 Continuous* wait a bit for scanners

2014-02-25 Thread Sean Busbey
On Feb. 25, 2014, 12:23 a.m., Sean Busbey wrote: Is there a disadvantage to instead updating the README to require creating the table ahead of time? That way we could avoid this code, and we could remove the table creation code from the writers. As a side effect, there'd also be a

Re: Review Request 18444: ACCUMULO-2399 Continuous* wait a bit for scanners

2014-02-25 Thread Mike Drob
On Feb. 25, 2014, 12:23 a.m., Sean Busbey wrote: Is there a disadvantage to instead updating the README to require creating the table ahead of time? That way we could avoid this code, and we could remove the table creation code from the writers. As a side effect, there'd also be a

Re: Review Request 18444: ACCUMULO-2399 Continuous* wait a bit for scanners

2014-02-25 Thread Sean Busbey
On Feb. 25, 2014, 12:23 a.m., Sean Busbey wrote: Is there a disadvantage to instead updating the README to require creating the table ahead of time? That way we could avoid this code, and we could remove the table creation code from the writers. As a side effect, there'd also be a

Review Request 18467: ACCUMULO-2404 Better error logging in start.Main

2014-02-25 Thread Mike Drob
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18467/ --- Review request for accumulo. Bugs: ACCUMULO-2404

Re: Review Request 18444: ACCUMULO-2399 Continuous* wait a bit for scanners

2014-02-25 Thread Bill Havanki
On Feb. 24, 2014, 7:23 p.m., Sean Busbey wrote: Is there a disadvantage to instead updating the README to require creating the table ahead of time? That way we could avoid this code, and we could remove the table creation code from the writers. As a side effect, there'd also be a

Re: Review Request 18467: ACCUMULO-2404 Better error logging in start.Main

2014-02-25 Thread Bill Havanki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18467/#review35424 --- Ship it! Ship It! - Bill Havanki On Feb. 25, 2014, 11:49 a.m.,

Review Request 18438: ACCUMULO-780

2014-02-25 Thread John McNamee
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18438/ --- Review request for accumulo. Repository: accumulo Description ---

Re: Review Request 18438: ACCUMULO-780

2014-02-25 Thread Mike Drob
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18438/#review35427 --- bin/bootstrap_config.sh

Re: Review Request 18438: ACCUMULO-780

2014-02-25 Thread John McNamee
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18438/ --- (Updated Feb. 25, 2014, 6:36 p.m.) Review request for accumulo. Changes

Re: Review Request 18444: ACCUMULO-2399 Alert user CI table should exist

2014-02-25 Thread Mike Drob
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18444/ --- (Updated Feb. 25, 2014, 8:46 p.m.) Review request for accumulo. Changes

Re: Review Request 18444: ACCUMULO-2399 Alert user CI table should exist

2014-02-25 Thread Sean Busbey
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18444/#review35467 --- Aside: it's also probably worth verifying that when you do create

Re: Disable Tracing

2014-02-25 Thread Sean Busbey
On Tue, Feb 25, 2014 at 2:16 PM, Mike Drob mad...@cloudera.com wrote: What is the recommended way to disable tracing on a cluster? I tried just killing my tracers, but now my tablet servers are spamming the monitor logs with ConnectionRefused exceptions. A cleaner method to do this would be

Re: Disable Tracing

2014-02-25 Thread Eric Newton
Try removing the tracer's entry in zookeeper. The spamming should have stopped when the ephemeral node advertising the tracer's location expired. -Eric On Tue, Feb 25, 2014 at 4:27 PM, Sean Busbey busbey+li...@cloudera.comwrote: On Tue, Feb 25, 2014 at 2:16 PM, Mike Drob mad...@cloudera.com

Re: [VOTE] Apache Accumulo 1.5.1-RC3

2014-02-25 Thread Keith Turner
I ran a utility [1] to analyze API diffs [2] between 1.5.0 and 1.5.1-RC3. The configs I used are the two xml files in the parent [3] of the report. I think the diff looks ok. I used jars from 1.5.0 and 1.5.1-RC3 bin.tar.gz. [1] : http://ispras.linuxbase.org/index.php/Java_API_Compliance_Checker

Re: [VOTE] Apache Accumulo 1.5.1-RC3

2014-02-25 Thread Adam Fuchs
Thanks for running that checker, Keith. Should we not be worried about the removal of InputFormatBase.RangeInputSplit? If I read correctly this will break both binary (runtime) compatibility and code (compile-time) compatibility. Can somebody make an argument for why this is not a problem in a

Re: [VOTE] Apache Accumulo 1.5.1-RC3

2014-02-25 Thread John Vines
Will it? Clients don't interact with that code at all directly. On Tue, Feb 25, 2014 at 5:57 PM, Adam Fuchs afu...@apache.org wrote: Thanks for running that checker, Keith. Should we not be worried about the removal of InputFormatBase.RangeInputSplit? If I read correctly this will break both

Re: [VOTE] Apache Accumulo 1.5.1-RC3

2014-02-25 Thread Josh Elser
I haven't checked what would happen. If you subclassed the RangeInputSplit, it's rather likely that you'd need a recompilation. On 2/25/14, 5:59 PM, John Vines wrote: Will it? Clients don't interact with that code at all directly. On Tue, Feb 25, 2014 at 5:57 PM, Adam Fuchs afu...@apache.org

Re: [VOTE] Apache Accumulo 1.5.1-RC3

2014-02-25 Thread Christopher
I don't know that this inner class used for M/R should be considered public API... nor do I imagine it will cause compatibility problems if users aren't referencing it in their code (which there's no reason to expect them to). I don't know if anybody is subclassing RangeInputSplit, but I'd suspect

Re: [DISCUSS] Updating planned release dates

2014-02-25 Thread Christopher
Seems reasonable to update these tentatively in JIRA... but I hope 1.6.0 is out before the end of March. I would say that the the 1.4.5 release will probably not benefit from 1.5.1's lessons learned much, though. A lot of work went into simplifying/automating the release process in 1.5.0 that I

Re: [VOTE] Apache Accumulo 1.5.1-RC3

2014-02-25 Thread Adam Fuchs
I'll buy that the RangeInputSplit is probably not referenced directly in user code. In this case it's probably not a big enough change to delay the release. Adam On Feb 25, 2014 6:19 PM, Christopher ctubb...@apache.org wrote: I don't know that this inner class used for M/R should be considered