accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mike Drob" <md...@mdrob.com>
Subject Re: Review Request 18444: ACCUMULO-2399 Alert user CI table should exist
Date Tue, 25 Feb 2014 20:46:08 GMT

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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message