[GitHub] sijie commented on issue #1344: Allow to configure read-only mode in ZooKeeperClient

2018-06-13 Thread GitBox
sijie commented on issue #1344: Allow to configure read-only mode in ZooKeeperClient URL: https://github.com/apache/bookkeeper/pull/1344#issuecomment-397061958 run integration tests This is an automated message from the

[GitHub] sijie commented on issue #1344: Allow to configure read-only mode in ZooKeeperClient

2018-06-11 Thread GitBox
sijie commented on issue #1344: Allow to configure read-only mode in ZooKeeperClient URL: https://github.com/apache/bookkeeper/pull/1344#issuecomment-396315700 run integration tests This is an automated message from the

[GitHub] sijie commented on issue #1344: Allow to configure read-only mode in ZooKeeperClient

2018-06-07 Thread GitBox
sijie commented on issue #1344: Allow to configure read-only mode in ZooKeeperClient URL: https://github.com/apache/bookkeeper/pull/1344#issuecomment-395619836 rerun bookkeeper-server bookie tests rerun integration tests

[GitHub] sijie commented on issue #1344: Allow to configure read-only mode in ZooKeeperClient

2018-06-07 Thread GitBox
sijie commented on issue #1344: Allow to configure read-only mode in ZooKeeperClient URL: https://github.com/apache/bookkeeper/pull/1344#issuecomment-395311322 fyi, I rebased this PR to latest master and fixed the compilation issue. waiting for CI to pass and will then merge it for

[GitHub] sijie commented on issue #1344: Allow to configure read-only mode in ZooKeeperClient

2018-05-15 Thread GitBox
sijie commented on issue #1344: Allow to configure read-only mode in ZooKeeperClient URL: https://github.com/apache/bookkeeper/pull/1344#issuecomment-389142663 @eolivelli retest doesn't help here, because there are some checkstyle errors in this PR.

[GitHub] sijie commented on issue #1344: Allow to configure read-only mode in ZooKeeperClient

2018-04-17 Thread GitBox
sijie commented on issue #1344: Allow to configure read-only mode in ZooKeeperClient URL: https://github.com/apache/bookkeeper/pull/1344#issuecomment-381869124 @merlimat it seems you need to fix tests This is an automated