Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1225#discussion_r185497944
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/coord/zk/TestZookeeperClient.java
---
@@ -125,7 +125,7 @@ public void testHasPathThrowsDrillRuntimeException() {
Mockito
.when(client.getCache().getCurrentData(absPath))
- .thenThrow(Exception.class);
+ .thenThrow(RuntimeException.class);
--- End diff --
IMO, this method needs to be changed to test
`ZookeeperClient.hasPath(String path, boolean consistent)`. It is OK to do it
in this PR or in a separate commit/JIRA/PR. If you decide to do it in a
separate commit, please file JIRA.
---
