curator-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [curator] Randgalt commented on a change in pull request #320: [CURATOR-533] - improve circuit breaking behavior
Date Thu, 08 Aug 2019 23:22:47 GMT
Randgalt commented on a change in pull request #320: [CURATOR-533] - improve circuit breaking
behavior
URL: https://github.com/apache/curator/pull/320#discussion_r312281525
 
 

 ##########
 File path: curator-framework/src/main/java/org/apache/curator/framework/state/ConnectionStateListener.java
 ##########
 @@ -31,14 +31,14 @@
     void stateChanged(CuratorFramework client, ConnectionState newState);
 
     /**
-     * Normally, ConnectionStateListeners are decorated via the configured
-     * {@link org.apache.curator.framework.state.ConnectionStateListenerDecorator}. For certain
-     * critical cases, however, this is not desired. If your listener returns <code>true</code>
-     * for doNotDecorate(), it will not be passed through the decorator.
+     * ConnectionStateListener managers set via {@link org.apache.curator.framework.CuratorFrameworkFactory.Builder#connectionStateListenerManagerFactory(ConnectionStateListenerManagerFactory)}
+     * are allowed to proxy (etc.) ConnectionStateListeners as needed. If this method returns
<code>true</code>
+     * the ConnectionStateListener manager must <em>not</em> proxy the listener
as it's a vital internal
+     * listener used by Curator.
      *
      * @return true/false
      */
-    default boolean doNotDecorate()
+    default boolean doNotProxy()
 
 Review comment:
   It’s not specific to circuit breaking. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message