kaisun2000 commented on pull request #1185: URL: https://github.com/apache/helix/pull/1185#issuecomment-669612859
This pull request is approved, please help to merge. >ZkHelixClusterVerifier and its subclass built with ZK addres would create an internal ZkClient. Without properly closing the ZkClient or using global ZkClient in the test base. There would be four threads and one ZooKeeper client object leaked. The leaked threads would be in thousands and thus be one of the main contributor preventing the unit test from finishing. This pull request address this issue. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
