ctubbsii commented on a change in pull request #341: ACCUMULO-3902 Ensure [Batch]Scanners are closed in ITs URL: https://github.com/apache/accumulo/pull/341#discussion_r157313951
########## File path: test/src/main/java/org/apache/accumulo/test/ClientSideIteratorIT.java ########## @@ -104,6 +104,7 @@ public void testIntersect() throws Exception { IntersectingIterator.setColumnFamilies(si, new Text[] {new Text("bar"), new Text("foo")}); csis.addScanIterator(si); + csis.close(); Review comment: Another case of it being closed before it is being used. I think switching to more try-with-resources syntax in these cases would fix a lot of these kinds of errors. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services