accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From els...@apache.org
Subject [2/3] git commit: ACCUMULO-3286 Get some more output on conditional writer. Don't have enough info as to why it failed.
Date Fri, 31 Oct 2014 22:45:30 GMT
ACCUMULO-3286 Get some more output on conditional writer. Don't have enough info as to why
it failed.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/09a4f0aa
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/09a4f0aa
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/09a4f0aa

Branch: refs/heads/master
Commit: 09a4f0aa6b47a99a27f6a0be76771e38e571f742
Parents: d3900b2
Author: Josh Elser <elserj@apache.org>
Authored: Fri Oct 31 17:41:30 2014 -0400
Committer: Josh Elser <elserj@apache.org>
Committed: Fri Oct 31 18:44:43 2014 -0400

----------------------------------------------------------------------
 .../org/apache/accumulo/test/ConditionalWriterIT.java   | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/09a4f0aa/test/src/test/java/org/apache/accumulo/test/ConditionalWriterIT.java
----------------------------------------------------------------------
diff --git a/test/src/test/java/org/apache/accumulo/test/ConditionalWriterIT.java b/test/src/test/java/org/apache/accumulo/test/ConditionalWriterIT.java
index 26c53bb..44765f1 100644
--- a/test/src/test/java/org/apache/accumulo/test/ConditionalWriterIT.java
+++ b/test/src/test/java/org/apache/accumulo/test/ConditionalWriterIT.java
@@ -1246,16 +1246,22 @@ public class ConditionalWriterIT extends SimpleMacIT {
           }
         }
       });
+      String traceOutput = finalBuffer.toString();
+      log.info("Trace output:" + traceOutput);
       if (traceCount > 0) {
         int lastPos = 0;
         for (String part : "traceTest, startScan,startConditionalUpdate,conditionalUpdate,Check
conditions,apply conditional mutations".split(","))
         {
-          int pos = finalBuffer.toString().indexOf(part);
-          assertTrue(pos > 0);
-          assertTrue(pos > lastPos);
+          log.info("Looking in trace output for '" + part + "'");
+          int pos = traceOutput.indexOf(part);
+          assertTrue("Did not find '" + part + "' in output", pos > 0);
+          assertTrue("'" + part + "' occurred earlier than the previous element unexpectedly",
pos > lastPos);
           lastPos = pos;
         }
         break;
+      } else {
+        log.info("Ignoring trace output as traceCount not greater than zero: " + traceCount);
+        Thread.sleep(1000);
       }
     }
     if (tracer != null) {


Mime
View raw message