Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/1234#discussion_r183608559 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/store/bson/TestBsonRecordReader.java --- @@ -272,6 +276,9 @@ public static void cleanUp() { } catch (Exception e) { } + + buffer.close(); --- End diff -- Currently close() just calls release(). I will update to use release(), but in general what is the best practice regarding when to call close() and when to call release()?
---