[accumulo] branch 1.7 updated: ACCUMULO-4847 Fix broken TabletServerGivesUpIT

[ct...@apache.org]

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

ctubbsii 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 db200c7  ACCUMULO-4847 Fix broken TabletServerGivesUpIT
db200c7 is described below

commit db200c7e9926851d9c2a3724a2c9576b685783df
Author: Christopher Tubbs <ct...@apache.org>
AuthorDate: Thu Mar 15 13:48:26 2018 -0400

    ACCUMULO-4847 Fix broken TabletServerGivesUpIT
    
    Fix retry behavior in TabletServerGivesUpIT by setting increment time to
    less than the max time, as now required in the Retry code. Also tweak
    timing / number of iterations for test retries.
---
 .../test/java/org/apache/accumulo/test/TabletServerGivesUpIT.java  | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/test/src/test/java/org/apache/accumulo/test/TabletServerGivesUpIT.java b/test/src/test/java/org/apache/accumulo/test/TabletServerGivesUpIT.java
index 68bd07b..7ca03d2 100644
--- a/test/src/test/java/org/apache/accumulo/test/TabletServerGivesUpIT.java
+++ b/test/src/test/java/org/apache/accumulo/test/TabletServerGivesUpIT.java
@@ -37,12 +37,13 @@ public class TabletServerGivesUpIT extends ConfigurableMacIT {
   public void configure(MiniAccumuloConfigImpl cfg, Configuration hadoopCoreSite) {
     cfg.useMiniDFS(true);
     cfg.setNumTservers(1);
-    cfg.setProperty(Property.INSTANCE_ZK_TIMEOUT, "15s");
-    cfg.setProperty(Property.TSERV_WALOG_TOLERATED_CREATION_FAILURES, "15");
+    cfg.setProperty(Property.INSTANCE_ZK_TIMEOUT, "10s");
+    cfg.setProperty(Property.TSERV_WALOG_TOLERATED_CREATION_FAILURES, "10");
+    cfg.setProperty(Property.TSERV_WALOG_TOLERATED_WAIT_INCREMENT, "0s");
     cfg.setProperty(Property.TSERV_WALOG_TOLERATED_MAXIMUM_WAIT_DURATION, "0s");
   }
 
-  @Test(timeout = 30 * 1000)
+  @Test(timeout = 45 * 1000)
   public void test() throws Exception {
     final Connector conn = this.getConnector();
     // Yes, there's a tabletserver

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