[GitHub] rdhabalia commented on issue #507: Add zk-stats instrumentation to get zk-client stats

2017-07-13 Thread git
rdhabalia commented on issue #507: Add zk-stats instrumentation to get zk-client stats URL: https://github.com/apache/incubator-pulsar/pull/507#issuecomment-315149273 let me check.. actually I have merged after [PR-travis](https://travis-ci.org/apache/incubator-pulsar/builds/252612895) bu

[GitHub] rdhabalia commented on issue #507: Add zk-stats instrumentation to get zk-client stats

2017-07-11 Thread git
rdhabalia commented on issue #507: Add zk-stats instrumentation to get zk-client stats URL: https://github.com/apache/incubator-pulsar/pull/507#issuecomment-314542547 @merlimat Addressed all the changes. This is an automated

[GitHub] rdhabalia commented on issue #507: Add zk-stats instrumentation to get zk-client stats

2017-07-10 Thread git
rdhabalia commented on issue #507: Add zk-stats instrumentation to get zk-client stats URL: https://github.com/apache/incubator-pulsar/pull/507#issuecomment-314287944 > What I was meaning is that since you mention the ZookeeperClient from BK, that wrapper is already collecting the latency

[GitHub] rdhabalia commented on issue #507: Add zk-stats instrumentation to get zk-client stats

2017-07-10 Thread git
rdhabalia commented on issue #507: Add zk-stats instrumentation to get zk-client stats URL: https://github.com/apache/incubator-pulsar/pull/507#issuecomment-314275283 > Can't the latency measurement be done in the ZK client wrapper? Latency is measured against [ZK-Client](https://gi