ACCUMULO-2194 Add delay for randomwalk Security teardown If two Security randomwalk tests run back-to-back, the second test may see that the table user still exists even though it was removed when the first test was torn down. This can happen if the user drop does not propagate through Zookeeper quickly enough. This commit adds a delay to the end of the Security test to give ZK some time.
Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/ea86b44d Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/ea86b44d Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/ea86b44d Branch: refs/heads/master Commit: ea86b44dfd6f46d389a630beb33ac49b65d87cd7 Parents: d11acbe Author: Bill Havanki <bhava...@cloudera.com> Authored: Wed Mar 12 10:51:06 2014 -0400 Committer: Bill Havanki <bhava...@cloudera.com> Committed: Thu Mar 13 09:30:09 2014 -0400 ---------------------------------------------------------------------- .../accumulo/server/test/randomwalk/security/SecurityFixture.java | 2 ++ 1 file changed, 2 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/accumulo/blob/ea86b44d/src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/SecurityFixture.java ---------------------------------------------------------------------- diff --git a/src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/SecurityFixture.java b/src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/SecurityFixture.java index c20e6db..144140b 100644 --- a/src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/SecurityFixture.java +++ b/src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/SecurityFixture.java @@ -94,5 +94,7 @@ public class SecurityFixture extends Fixture { log.debug("Dropping user: " + systemUserName); conn.securityOperations().dropUser(systemUserName); + // Allow user drops to propagate, in case a new security test starts + Thread.sleep(2000); } }
