narendly commented on issue #3894: Replace deprecated ZkClient APIs URL: https://github.com/apache/incubator-pinot/pull/3894#issuecomment-468167263 My analysis in the log above was incorrect. There's no change to the commit itself, but I've identified the exact cause for this. See the following: The underlying interface for ZkClient in HelixManager has been changed to HelixZkClient. Using ZkClient directly is no longer recommended by Helix because it might cause ZkConnection leakage. However, the way Pinot uses ZkClient here is for testing purposes only, so we need to just replace the deprecated APIs with updated ones.
---------------------------------------------------------------- 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 --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
