Github user asfgit closed the pull request at:
https://github.com/apache/drill/pull/1023
---
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
Github user ilooner closed the pull request at:
https://github.com/apache/drill/pull/1023
---
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r159131381
--- Diff: pom.xml ---
@@ -442,7 +442,7 @@
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r155938262
--- Diff: common/src/test/java/org/apache/drill/test/DrillTest.java ---
@@ -54,7 +54,7 @@
static MemWatcher memWatcher;
static String
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r155938285
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -61,12 +63,14 @@
public class WorkManager implements
Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r150348155
--- Diff: pom.xml ---
@@ -442,7 +442,7 @@
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r150345195
--- Diff: pom.xml ---
@@ -442,7 +442,7 @@
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r150339444
--- Diff: pom.xml ---
@@ -442,7 +442,7 @@
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r150336286
--- Diff: pom.xml ---
@@ -442,7 +442,7 @@
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r150327067
--- Diff: pom.xml ---
@@ -442,7 +442,7 @@
Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r150322554
--- Diff: pom.xml ---
@@ -442,7 +442,7 @@
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r150302692
--- Diff: pom.xml ---
@@ -442,7 +442,7 @@
Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r150259975
--- Diff: pom.xml ---
@@ -442,7 +442,7 @@
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149257480
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149257347
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149257392
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149230560
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149234360
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149232323
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149173898
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/user/PlanSplitter.java
---
@@ -79,6 +80,15 @@ public QueryPlanFragments
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149168826
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149168081
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java ---
@@ -828,8 +828,13 @@ public void close() throws Exception {
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149167784
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149167742
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149167716
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r149166078
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -59,12 +61,14 @@
public class WorkManager implements
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148997365
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148997565
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java ---
@@ -828,8 +828,13 @@ public void close() throws Exception {
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148997959
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148997637
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/user/PlanSplitter.java
---
@@ -79,6 +80,15 @@ public QueryPlanFragments
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148998119
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,49 @@ public DrillbitContext getContext() {
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148997788
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -59,12 +61,14 @@
public class WorkManager implements
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148896551
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,51 @@ public DrillbitContext getContext() {
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148894504
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,51 @@ public DrillbitContext getContext() {
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148894158
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,51 @@ public DrillbitContext getContext() {
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148893046
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,51 @@ public DrillbitContext getContext() {
Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148892741
--- Diff: pom.xml ---
@@ -442,7 +442,7 @@
Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148891238
--- Diff: pom.xml ---
@@ -442,7 +442,7 @@
Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148889490
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,51 @@ public DrillbitContext getContext() {
Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148890716
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,51 @@ public DrillbitContext getContext() {
Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r14668
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,51 @@ public DrillbitContext getContext() {
Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1023#discussion_r148889092
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java ---
@@ -158,38 +165,51 @@ public DrillbitContext getContext() {
43 matches
Mail list logo