-----------------------------------------------------------
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
-------

Instead of sleeping, have the continuous clients output a slightly more 
informative error message. Remove the table creation code path from continuous 
ingest.


Summary (updated)
-----------------

ACCUMULO-2399 Alert user CI table should exist


Bugs: ACCUMULO-2399
    https://issues.apache.org/jira/browse/ACCUMULO-2399


Repository: accumulo


Description (updated)
-------

ACCUMULO-2399 Alert user CI table should exist

Extract common table checking functionality for continuous clients and
fail fast when the table does not exist. No longer create the table in
ingest to make the requirement explicit.


Diffs (updated)
-----

  
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/ContinuousIngest.java
 b63efbc9c7423be8985a292d0c162a672e4b65a0 
  
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 (updated)
-------

Unit tests. Started continuous ingest against a local cluster and it did not 
run.


Thanks,

Mike Drob

Reply via email to