Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1213#discussion_r182444596
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/vector/complex/writer/TestPromotableWriter.java
---
@@ -33,6 +33,7 @@
public void list() throws Exception {
BufferAllocator allocator =
RootAllocatorFactory.newRoot(DrillConfig.create());
TestOutputMutator output = new TestOutputMutator(allocator);
+ @SuppressWarnings("resource")
--- End diff --
Why is it necessary to suppress resource not being closed warning here and
OK to remove warning suppression in other places?
---