Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/curator/pull/262#discussion_r179652781
--- Diff:
curator-framework/src/main/java/org/apache/curator/framework/state/ConnectionStateManager.java
---
@@ -340,4 +338,12 @@ private void setCurrentConnectionState(ConnectionState
newConnectionState)
currentConnectionState = newConnectionState;
startOfSuspendedEpoch = (currentConnectionState ==
ConnectionState.SUSPENDED) ? System.currentTimeMillis() : 0;
}
+
+ private synchronized int getUseSessionTimeoutMs() {
--- End diff --
I think volatile is cleaner. I don't see any read/increments so it doesn't
need to be AtomicLong either.
---
