Github user asfgit closed the pull request at:
https://github.com/apache/drill/pull/1073
---
Github user ilooner closed the pull request at:
https://github.com/apache/drill/pull/1073
---
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 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 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 void
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 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 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 testA
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 m
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 wa
10 matches
Mail list logo