Sigma-Ma commented on PR #8556: URL: https://github.com/apache/hbase/pull/8556#issuecomment-5380569791
> This was intentionally added by @saintstack here [7df9490](https://github.com/apache/hbase/commit/7df9490d60d770b87a480a232c8edf166e0629d2). Although I didn't get it, he can help us here. Thanks for pointing this out. I checked the original commit, and its message says the TestZooKeeper change was “probably useless” and that the actual issue was related to the ZooKeeper startup check hanging. The test already starts a MiniDFSCluster earlier in setUpBeforeClass, so the added line starts it twice on the same TEST_UTIL. That’s why this PR removes only the second call. @saintstack, please let me know if I’m missing any context here. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
