[GitHub] drill pull request #1225: DRILL-6272: Refactor dynamic UDFs and function ini...

[vrozov]

Github user vrozov commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1225#discussion_r185369981
  
    --- 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 --
    
    OK, but I am not sure what does this method test. 
`ZookeeperClient.hasPath(String path)` is not used in production.


---