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.
---