Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/curator/pull/266#discussion_r197641440
--- Diff: curator-client/src/main/java/org/apache/curator/HandleHolder.java
---
@@ -73,15 +73,15 @@ String getNewConnectionString()
return ((helperConnectionString != null) &&
!ensembleProvider.getConnectionString().equals(helperConnectionString)) ?
helperConnectionString : null;
}
- void closeAndClear() throws Exception
+ void closeAndClear(int timeout) throws Exception
--- End diff --
Please rename argument to `waitForShutdownTimeoutMs`
---
