[GitHub] drill pull request #1073: DRILL-5967: Fixed memory leak in OrderedPartitionS...

2018-01-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/1073 ---

[GitHub] drill pull request #1073: DRILL-5967: Fixed memory leak in OrderedPartitionS...

2018-01-11 Thread ilooner
Github user ilooner closed the pull request at: https://github.com/apache/drill/pull/1073 ---

[GitHub] drill pull request #1073: DRILL-5967: Fixed memory leak in OrderedPartitionS...

2018-01-11 Thread ilooner
GitHub user ilooner reopened a pull request: https://github.com/apache/drill/pull/1073 DRILL-5967: Fixed memory leak in OrderedPartitionSender The OrderedPartitionSender was leaking memory every time it was created because it created a wrapper RecordBatch which allocated memory but

[GitHub] drill pull request #1073: DRILL-5967: Fixed memory leak in OrderedPartitionS...

2017-12-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1073#discussion_r158591666 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionSenderRootExec.java --- @@ -341,6 +351,10 @@ public

[GitHub] drill pull request #1073: DRILL-5967: Fixed memory leak in OrderedPartitionS...

2017-12-22 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/1073#discussion_r158577033 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionSenderRootExec.java --- @@ -341,6 +351,10 @@ public

[GitHub] drill pull request #1073: DRILL-5967: Fixed memory leak in OrderedPartitionS...

2017-12-20 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1073#discussion_r158158542 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionSenderRootExec.java --- @@ -341,6 +351,10 @@ public

[GitHub] drill pull request #1073: DRILL-5967: Fixed memory leak in OrderedPartitionS...

2017-12-18 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1073#discussion_r157605837 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionSenderCreator.java --- @@ -35,7 +35,5 @@ public RootExec

[GitHub] drill pull request #1073: DRILL-5967: Fixed memory leak in OrderedPartitionS...

2017-12-18 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1073#discussion_r157605488 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/partitionsender/TestPartitionSender.java --- @@ -350,7 +350,7 @@ public void

[GitHub] drill pull request #1073: DRILL-5967: Fixed memory leak in OrderedPartitionS...

2017-12-18 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1073#discussion_r157545737 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionSenderRootExec.java --- @@ -98,10 +100,12 @@ public int

[GitHub] drill pull request #1073: DRILL-5967: Fixed memory leak in OrderedPartitionS...

2017-12-14 Thread ilooner
GitHub user ilooner opened a pull request: https://github.com/apache/drill/pull/1073 DRILL-5967: Fixed memory leak in OrderedPartitionSender The OrderedPartitionSender was leaking memory every time it was created because it created a wrapper RecordBatch which allocated memory but