Skip to content

Conversation

@kezhuw
Copy link
Member

@kezhuw kezhuw commented Jul 3, 2025

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);
}

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);
}
```
@kezhuw kezhuw requested review from Randgalt, eolivelli and tisonkun July 3, 2025 15:21
@kezhuw
Copy link
Member Author

kezhuw commented Jul 3, 2025

See also #1252 (review).

@kezhuw kezhuw merged commit 7123060 into apache:master Jul 4, 2025
10 checks passed
@kezhuw kezhuw deleted the fix-flaky-TestReadOnly.testConnectionStateNewClient branch August 18, 2025 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants