Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1060#discussion_r158181731
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SystemOptionManager.java
 ---
    @@ -449,7 +451,7 @@ public void close() throws Exception {
         // been created. Gracefully handle that case.
     
         if (options != null) {
    -      options.close();
    -    }
    +    options.close();
       }
     }
    +}
    --- End diff --
    
    Drill kind of likes the closing bracket to align with the opening 
statement, and the body to be indented two spaces. At least in Git, the 
indentation here seems off from the Drill practice.


---

Reply via email to