You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ec...@apache.org on 2012/12/19 20:57:20 UTC

svn commit: r1424060 - in /accumulo/trunk: fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooReader.java server/src/main/java/org/apache/accumulo/server/test/functional/BulkSplitOptimizationTest.java test/system/auto/TestUtils.py

Author: ecn
Date: Wed Dec 19 19:57:20 2012
New Revision: 1424060

URL: http://svn.apache.org/viewvc?rev=1424060&view=rev
Log:
ACCUMULO-408 get the bulk functional tests working

Modified:
    accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooReader.java
    accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/functional/BulkSplitOptimizationTest.java
    accumulo/trunk/test/system/auto/TestUtils.py

Modified: accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooReader.java
URL: http://svn.apache.org/viewvc/accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooReader.java?rev=1424060&r1=1424059&r2=1424060&view=diff
==============================================================================
--- accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooReader.java (original)
+++ accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooReader.java Wed Dec 19 19:57:20 2012
@@ -18,6 +18,7 @@ package org.apache.accumulo.fate.zookeep
 
 import java.util.List;
 import java.util.concurrent.atomic.AtomicBoolean;
+import java.util.concurrent.atomic.AtomicInteger;
 
 import org.apache.zookeeper.KeeperException;
 import org.apache.zookeeper.Watcher;
@@ -76,12 +77,12 @@ public class ZooReader implements IZooRe
   
   @Override
   public void sync(final String path) throws KeeperException, InterruptedException {
-    final int[] rc = { 0 };
+    final AtomicInteger rc = new AtomicInteger();
     final AtomicBoolean waiter = new AtomicBoolean(false);
     getZooKeeper().sync(path, new VoidCallback() {
       @Override
       public void processResult(int code, String arg1, Object arg2) {
-        rc[0] = code;
+        rc.set(code);
         synchronized (waiter) {
           waiter.set(true);
           waiter.notifyAll();
@@ -91,12 +92,11 @@ public class ZooReader implements IZooRe
       while (!waiter.get())
         waiter.wait();
     }
-    Code code = Code.get(rc[0]);
+    Code code = Code.get(rc.get());
     if (code != KeeperException.Code.OK) {
       throw KeeperException.create(code);
     }
-  }
-  
+  }  
   
   public ZooReader(String keepers, int timeout) {
     this.keepers = keepers;

Modified: accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/functional/BulkSplitOptimizationTest.java
URL: http://svn.apache.org/viewvc/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/functional/BulkSplitOptimizationTest.java?rev=1424060&r1=1424059&r2=1424060&view=diff
==============================================================================
--- accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/functional/BulkSplitOptimizationTest.java (original)
+++ accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/functional/BulkSplitOptimizationTest.java Wed Dec 19 19:57:20 2012
@@ -63,7 +63,7 @@ public class BulkSplitOptimizationTest e
     FileSystem fs = FileSystem.get(CachedConfiguration.getInstance());
     fs.delete(new Path("/tmp/testmf"), true);
     
-    CreateRFiles.main(new String[] {"tmp/testmf", "8", "0", "100000", "99"});
+    CreateRFiles.main(new String[] { "--output", "tmp/testmf", "--numThreads", "8", "--start", "0", "--end", "100000", "--splits", "99"});
     
     bulkImport(fs, TABLE_NAME, "/tmp/testmf");
     
@@ -82,7 +82,7 @@ public class BulkSplitOptimizationTest e
     
     checkSplits(TABLE_NAME, 50, 100);
     
-    VerifyIngest.main(new String[] {"-timestamp", "1", "-size", "50", "-random", "56", "100000", "0", "1"});
+    VerifyIngest.main(new String[] {"--timestamp", "1", "--size", "50", "--random", "56", "--rows", "100000", "--start", "0", "--cols", "1"});
     
     // ensure each tablet does not have all map files
     checkRFiles(TABLE_NAME, 50, 100, 1, 4);

Modified: accumulo/trunk/test/system/auto/TestUtils.py
URL: http://svn.apache.org/viewvc/accumulo/trunk/test/system/auto/TestUtils.py?rev=1424060&r1=1424059&r2=1424060&view=diff
==============================================================================
--- accumulo/trunk/test/system/auto/TestUtils.py (original)
+++ accumulo/trunk/test/system/auto/TestUtils.py Wed Dec 19 19:57:20 2012
@@ -52,6 +52,7 @@ General_CLASSPATH = """
     $ACCUMULO_HOME/server/target/classes/,
     $ACCUMULO_HOME/core/target/classes/,
     $ACCUMULO_HOME/start/target/classes/,
+    $ACCUMULO_HOME/fate/target/classes/,
     $ACCUMULO_HOME/examples/instamo/target/classes,
     $ACCUMULO_HOME/examples/simple/target/classes,
     $ACCUMULO_HOME/lib/[^.].$ACCUMULO_VERSION.jar,