Signed-off-by: tison <[email protected]>

This closes #1246 and closes #1247.

…k::list (#1257)

#1250 introduced a bug where CachedModeledFramework::list is no longer returning all descendants of the basePath of the underlying cache and is instead returning the direct children of the current path of the client.

Closes #1254.

Closes #1258.

See also CURATOR-545, ZOOKEEPER-3546(apache/zookeeper#1138).

`NamespaceFacade` does not support `getCuratorListenable` while #520 use
it to listen for `CuratorEventType.CLOSING` to fix CURATOR-729.

This commit exports `CuratorFrameworkBase::client` to retrieve
underlying framework client to listen for for `CuratorEventType.CLOSING`.

Fixes #1259.

Fixes #1263.

Closes #1251.

Co-authored-by: Amina Dinari <[email protected]>
Co-authored-by: Kezhu Wang <[email protected]>

`testAdd`, `testAddAsync` and `testAddAndRemove` asserts that
`EnsembleProvider::setConnectionString` is updated with new connect
string after ensemble changed. Since the update is asynchronous, so they
should wait before assertion.

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