jira-importer commented on issue #528:
URL: https://github.com/apache/curator/issues/528#issuecomment-2604691263

   <i><a 
href="https://issues.apache.org/jira/secure/ViewProfile.jspa?name=arren";>arren</a>:</i>
   <p>ConnectionState.checkTimeouts resets the connection if the session 
timeout has expired, whether or not we ever had a session. If one has a large, 
shooty cluster with a short session timeout and a long connection timeout, this 
might reset the host list pointer to zero every sessionTimeoutMs, never letting 
it connect to a valid, available host towards the end of the list. On the other 
hand, if it is an assertion that sessionTimeout > connectionTimeout always, 
then the min() call is not required and the ifelse should read "else if 
(elapsed > connectionTimeoutMs)" instead of just "else".</p>


-- 
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.

To unsubscribe, e-mail: dev-unsubscr...@curator.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to