Github user keith-turner commented on a diff in the pull request:
https://github.com/apache/curator/pull/267#discussion_r194743349
--- Diff:
curator-framework/src/main/java/org/apache/curator/framework/imps/CuratorFrameworkImpl.java
---
@@ -402,75 +402,72 @@ public String getNamespace()
return (str != null) ? str : "";
}
+ private void checkState()
+ {
+ CuratorFrameworkState state = getState();
+ Preconditions.checkState(state == CuratorFrameworkState.STARTED,
"instance state must be %s before calling this method, however its %s",
CuratorFrameworkState.STARTED, state);
--- End diff --
Could make it less wordy, like : `Expected state %s, however it was %s`
---
