Github user sachouche commented on a diff in the pull request:
https://github.com/apache/drill/pull/1060#discussion_r161030963
--- 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 --
The issue was with the close() method which was not properly indented.
Fixed it.
---
