[GitHub] sohami commented on a change in pull request #1600: DRILL-6947: fix RuntimeFilter memory leak

2019-01-23 Thread GitBox
sohami commented on a change in pull request #1600: DRILL-6947: fix RuntimeFilter memory leak URL: https://github.com/apache/drill/pull/1600#discussion_r250298481 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/filter/RuntimeFilterRecordBatch.java ##

[GitHub] sohami commented on a change in pull request #1600: DRILL-6947: fix RuntimeFilter memory leak

2019-01-22 Thread GitBox
sohami commented on a change in pull request #1600: DRILL-6947: fix RuntimeFilter memory leak URL: https://github.com/apache/drill/pull/1600#discussion_r249937518 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/HashJoinBatch.java ## @@

[GitHub] sohami commented on a change in pull request #1600: DRILL-6947: fix RuntimeFilter memory leak

2019-01-22 Thread GitBox
sohami commented on a change in pull request #1600: DRILL-6947: fix RuntimeFilter memory leak URL: https://github.com/apache/drill/pull/1600#discussion_r249888648 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentContextImpl.java ## @@ -530,11 +

[GitHub] sohami commented on a change in pull request #1600: DRILL-6947: fix RuntimeFilter memory leak

2019-01-22 Thread GitBox
sohami commented on a change in pull request #1600: DRILL-6947: fix RuntimeFilter memory leak URL: https://github.com/apache/drill/pull/1600#discussion_r249937947 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/filter/RuntimeFilterRecordBatch.java ##