[GitHub] drill pull request #989: DRILL-5790: PCAP format explicitly opens local file

[arina-ielchiieva]

Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/989#discussion_r144269017
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/store/pcap/PcapRecordReader.java
 ---
    @@ -125,8 +129,8 @@ public int next() {
     
       @Override
       public void close() throws Exception {
    -//    buffer = null;
    -//    in.close();
    +    in.close();
    +    fs.close();
    --- End diff --
    
    Since you did not open fs, I guess it's not your responsibility to close it.


---