This is an automated email from the ASF dual-hosted git repository.

mmiller pushed a commit to branch 1.7
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/1.7 by this push:
     new bcd2dc5  Fix false positive in rw concurrent/AddSplits
bcd2dc5 is described below

commit bcd2dc51427a92474b9675cd03c156861308d365
Author: Mike Miller <mmil...@apache.org>
AuthorDate: Wed Feb 28 13:24:00 2018 -0500

    Fix false positive in rw concurrent/AddSplits
---
 .../org/apache/accumulo/test/randomwalk/concurrent/AddSplits.java  | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git 
a/test/src/main/java/org/apache/accumulo/test/randomwalk/concurrent/AddSplits.java
 
b/test/src/main/java/org/apache/accumulo/test/randomwalk/concurrent/AddSplits.java
index dc040a6..e026299 100644
--- 
a/test/src/main/java/org/apache/accumulo/test/randomwalk/concurrent/AddSplits.java
+++ 
b/test/src/main/java/org/apache/accumulo/test/randomwalk/concurrent/AddSplits.java
@@ -22,6 +22,7 @@ import java.util.Properties;
 import java.util.Random;
 import java.util.TreeSet;
 
+import org.apache.accumulo.core.client.AccumuloException;
 import org.apache.accumulo.core.client.Connector;
 import org.apache.accumulo.core.client.TableNotFoundException;
 import org.apache.accumulo.core.client.TableOfflineException;
@@ -57,6 +58,12 @@ public class AddSplits extends Test {
       log.debug("AddSplits " + tableName + " failed, doesnt exist");
     } catch (TableOfflineException e) {
       log.debug("AddSplits " + tableName + " failed, offline");
+    } catch (AccumuloException ae) {
+      Throwable cause = ae.getCause();
+      if (cause != null && cause instanceof TableOfflineException)
+        log.debug("AddSplits " + tableName + " failed, offline");
+      else
+        throw ae;
     }
   }
 }

-- 
To stop receiving notification emails like this one, please contact
mmil...@apache.org.

Reply via email to