[CURATOR-472] Test-only code started to be called in prod

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Fix Version/s: 4.1.0
• Component/s: Client
• Assignee: Jordan Zimmerman
• Reporter: mskonovalov
• Created: 23/Jul/18 05:31
• Updated: 03/Dec/18 01:18
• Resolved: 03/Dec/18 01:18

Description

After commit fe2c7c4cd606c0cf4bc4fab15deedc0f4c33ea0e the method

 

protected boolean testableWaitForShutdown(int wait)
    throws InterruptedException

with the following comment

 

 

* Wait up to wait milliseconds for the underlying threads to shutdown.
* THIS METHOD IS EXPECTED TO BE USED FOR TESTING ONLY!!!

started to be called from real method ZooKeeper.close()

It prevents tests to finish.

 

Issue Links

• links to: GitHub Pull Request #283