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

Reply via email to