GitHub user ilooner reopened a pull request:
https://github.com/apache/drill/pull/1023
DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 Stabalize TestValueVector
tests.
## DRILL-5922
- QueryContext was never closed when the Foreman finished. So the query
child allocator was never closed.
- The PlanSplitter created a QueryContext temporarily to construct an RPC
message but never closed it.
- The waitForExit method was error prone. Changed it to use the standard
condition variable pattern.
## DRILL-5926
The TestValueVector tests would run out of memory. I simple increased the
MaxDirectMemorySize for the forked test processes in the pom to avoid this.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ilooner/drill DRILL-5922
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/drill/pull/1023.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 #1023
----
commit 005a478238ff8a0ec085e051c099a109a73d3e8a
Author: Timothy Farkas <timothyfarkas@...>
Date: 2017-11-10T19:13:18Z
DRILL-5922:
- The QueryContext was never closed when the Foreman finished, so it's
child allocator was never closed. Now it is.
- The PlanSplitter created a QueryContext temporarily to construct an RPC
message but never closed it. Now the temp QueryContext is closed.
- The waitForExit method was error prone. Changed it to use the standard
condition variable pattern.
- Fixed timeouts in graceful shutdown tests
commit 551c54eca5ed9a69b990daa2a300c850e11e9376
Author: Timothy Farkas <timothyfarkas@...>
Date: 2017-11-10T20:59:42Z
DRILL-5926: The TestValueVector tests would run out of memory. Increased
the MaxDirectMemorySize for the forked test processes in the pom to avoid this.
commit 31175885bcd60abde5ec88ff3fcece589619265f
Author: Timothy Farkas <timothyfarkas@...>
Date: 2018-01-09T09:20:09Z
DRILL-6003: Fixed sporadic test failures caused by the creation of
duplicate clusters in TestDynamicUDFSupport.
----
---
