[curator] branch master updated: CURATOR-567 - At this point, TestCleanState is so flakey we should just turn it off for now. It's not serving much purpose anyway.

[randgalt]

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

randgalt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/curator.git


The following commit(s) were added to refs/heads/master by this push:
     new 4fb8f2c  CURATOR-567 - At this point, TestCleanState is so flakey we 
should just turn it off for now. It's not serving much purpose anyway.
     new 629486f  Merge branch 'CURATOR-567-remove-test-clean-state-where-flaky'
4fb8f2c is described below

commit 4fb8f2cf1ff476dffa38eaba7fff927e528ee212
Author: randgalt <randg...@apache.org>
AuthorDate: Sun Apr 19 18:26:13 2020 -0500

    CURATOR-567 - At this point, TestCleanState is so flakey we should just 
turn it off for now. It's not serving much purpose anyway.
---
 curator-client/src/main/java/org/apache/curator/utils/DebugUtils.java | 1 +
 .../test/java/org/apache/curator/framework/imps/TestCleanState.java   | 4 +++-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git 
a/curator-client/src/main/java/org/apache/curator/utils/DebugUtils.java 
b/curator-client/src/main/java/org/apache/curator/utils/DebugUtils.java
index 45eea22..7ac4c4d 100644
--- a/curator-client/src/main/java/org/apache/curator/utils/DebugUtils.java
+++ b/curator-client/src/main/java/org/apache/curator/utils/DebugUtils.java
@@ -26,6 +26,7 @@ public class DebugUtils
     public static final String 
PROPERTY_LOG_ONLY_FIRST_CONNECTION_ISSUE_AS_ERROR_LEVEL = 
"curator-log-only-first-connection-issue-as-error-level";
     public static final String PROPERTY_REMOVE_WATCHERS_IN_FOREGROUND = 
"curator-remove-watchers-in-foreground";
     public static final String PROPERTY_VALIDATE_NAMESPACE_WATCHER_MAP_EMPTY = 
"curator-validate-namespace-watcher-map-empty";
+    public static final String PROPERTY_VALIDATE_NO_REMAINING_WATCHERS = 
"curator-validate-no-remaining-watchers";
 
 
     private DebugUtils()
diff --git 
a/curator-framework/src/test/java/org/apache/curator/framework/imps/TestCleanState.java
 
b/curator-framework/src/test/java/org/apache/curator/framework/imps/TestCleanState.java
index b4791ff..ae1102e 100644
--- 
a/curator-framework/src/test/java/org/apache/curator/framework/imps/TestCleanState.java
+++ 
b/curator-framework/src/test/java/org/apache/curator/framework/imps/TestCleanState.java
@@ -27,9 +27,11 @@ import java.util.concurrent.Callable;
 
 public class TestCleanState
 {
+    private static final boolean IS_ENABLED = 
Boolean.getBoolean("PROPERTY_VALIDATE_NO_REMAINING_WATCHERS");
+
     public static void closeAndTestClean(CuratorFramework client)
     {
-        if ( client == null )
+        if ( (client == null) || !IS_ENABLED )
         {
             return;
         }