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

    https://github.com/apache/drill/pull/1145#discussion_r172667685
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/client/DrillClient.java ---
    @@ -312,6 +312,11 @@ public synchronized void connect(String connect, 
Properties props) throws RpcExc
         if (connected) {
           return;
         }
    +
    +    if (props == null) {
    --- End diff --
    
    Consider making `props` `@NotNull`.


---

Reply via email to