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 was never
closed.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ilooner/drill DRILL-5967
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/drill/pull/1073.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1073
----
commit 13cd2fe7e8dccb2ac7546de508fbfbed8e19b48b
Author: Timothy Farkas <timothyfarkas@...>
Date: 2017-12-14T18:48:27Z
DRILL-5967: Fixed memory leak in OrderedPartitionSender
commit e48e274649be7b9a14746c200d0ec5f5b3740664
Author: Timothy Farkas <timothyfarkas@...>
Date: 2017-12-18T20:01:30Z
- Applied review comments
commit 2233e6b32c1dd9f1b89fe4d6fa557e8bd894592e
Author: Timothy Farkas <timothyfarkas@...>
Date: 2017-12-18T21:54:01Z
- Applied more review comments
commit f3aef285fc168cad07b32e5e99cd2cd7811d765c
Author: Timothy Farkas <timothyfarkas@...>
Date: 2017-12-18T21:56:13Z
- Applied review comments
commit 0cdb2f3c4944f6253c72d318276603fc329546b5
Author: Timothy Farkas <timothyfarkas@...>
Date: 2017-12-19T19:27:19Z
- Removed unnecessary creation of a list in OrderedPartitionSenderCreator
commit f6d4aec39aad7326a735b883647772b28e2b86ab
Author: Timothy Farkas <timothyfarkas@...>
Date: 2018-01-11T22:19:18Z
- Added comment documenting the workaround.
----
---
