kezhuw opened a new pull request, #1272:
URL: https://github.com/apache/curator/pull/1272
ZOOKEEPER-4508 reports `SessionExpired` after exhausting session timeout
in session establishment instead of endless retries. It is easy to get
in this now with small session timeout.
Besides above, `sessionTimeoutMs` should bigger than `connectionTimeoutMs`,
while default `Timing::session()` and `Timing::connection()` are `2s` and
`10s` respectively now.
```
// from CuratorZookeeperClient
if (sessionTimeoutMs < connectionTimeoutMs) {
log.warn(
"session timeout [{}] is less than connection timeout [{}]",
sessionTimeoutMs, connectionTimeoutMs);
}
```
--
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: commits-unsubscr...@curator.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
