Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1225#discussion_r185449640
  
    --- 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 --
    
    I just fixed this test to work with new mockito lib version. I suspect this 
test checks if exception thrown from 
`client.getCache().getCurrentData(absPath)` method will be wrapped in 
DrillRuntimeException.


---

Reply via email to