merlimat commented on issue #507: Add zk-stats instrumentation to get zk-client stats URL: https://github.com/apache/incubator-pulsar/pull/507#issuecomment-314289298 I know, that's why I was saying whether it was easier to just import the code, until we sync up with future BK releases :) -- Matteo Merli <[email protected]> On Mon, Jul 10, 2017 at 5:29 PM, Rajan Dhabalia <[email protected]> wrote: > What I was meaning is that since you mention the ZookeeperClient from BK, > that wrapper is already collecting the latency stats (through the > StatsLogger mechanism). > > OpStatsLogger > <https://github.com/apache/bookkeeper/blob/master/bookkeeper-server/src/main/java/org/apache/bookkeeper/zookeeper/ZooWorker.java#L45> > is not present into BK-Yahoo version > <https://github.com/yahoo/bookkeeper/blob/yahoo-4.3/bookkeeper-server/src/main/java/org/apache/bookkeeper/zookeeper/ZooWorker.java> > > ? > You are receiving this because your review was requested. > Reply to this email directly, view it on GitHub > <https://github.com/apache/incubator-pulsar/pull/507#issuecomment-314287944>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/AAD0JPZ8ODbPwZi9Sz61Dmx2NK-dJS0cks5sMsHhgaJpZM4OARCW> > . > ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
With regards, Apache Git Services
