> 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 set where the tester would implicitly > > confirm that the user/password for accessing Accumulo works and can deal > > with the table. > > Mike Drob wrote: > The README already instructs the user to create the table before running. > It could be made more clear, though. > > Sean Busbey wrote: > If the README already tells the user to create the table all the more > reason to enforce following the instructions. > > Mike Drob wrote: > So you suggest removing the table creation code path from ingest? I'd be > ok with that.
yeah. maybe update the error handling in both ingest and the reader clients so that when the table doesn't exists it says to go back to the instructions on creating the table. - Sean ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18444/#review35352 ----------------------------------------------------------- On Feb. 24, 2014, 11:35 p.m., Mike Drob wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18444/ > ----------------------------------------------------------- > > (Updated Feb. 24, 2014, 11:35 p.m.) > > > Review request for accumulo. > > > Bugs: ACCUMULO-2399 > https://issues.apache.org/jira/browse/ACCUMULO-2399 > > > Repository: accumulo > > > Description > ------- > > ACCUMULO-2399 Continuous* wait a bit for scanners > > When starting ContinuousIngest concurrently with consumers, many of the > consumers would die if they came up before the table had been created. > Make the code a bit more robust during start-up. > > > Diffs > ----- > > > src/server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousBatchWalker.java > 4659affc49fc7fc2519e092330dda817ccbbadcd > > src/server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousQuery.java > c8ae6ecf9e5ef2bd51cb803cba429baaf57f3d6a > > src/server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousScanner.java > 0ac3df63a260a201544b3623d4901b355d7ea21a > > src/server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousUtil.java > PRE-CREATION > > src/server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousWalk.java > 7d1e7f95c7e5a7a172990322e2992e536f2725e3 > > Diff: https://reviews.apache.org/r/18444/diff/ > > > Testing > ------- > > Unit tests. Started processes on 3 node cluster and batch walkers did not die. > > > Thanks, > > Mike Drob > >
