[GitHub] drill pull request #1213: Minor code cleanup

[vrozov]

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?


---