[GitHub] [druid] clintropolis commented on a change in pull request #10076: ensure ParallelMergeCombiningSequence closes it's closeables

2020-06-24 Thread GitBox
clintropolis commented on a change in pull request #10076: URL: https://github.com/apache/druid/pull/10076#discussion_r445326768 ## File path: core/src/main/java/org/apache/druid/java/util/common/guava/ParallelMergeCombiningSequence.java ## @@ -1036,11 +1047,13 @@ public boole

[GitHub] [druid] clintropolis commented on a change in pull request #10076: ensure ParallelMergeCombiningSequence closes it's closeables

2020-06-26 Thread GitBox
clintropolis commented on a change in pull request #10076: URL: https://github.com/apache/druid/pull/10076#discussion_r446363639 ## File path: core/src/main/java/org/apache/druid/java/util/common/guava/ParallelMergeCombiningSequence.java ## @@ -341,6 +344,7 @@ protected void c