Re: [PR] DRILL-8479: Merge Join Leak When Depleting Incoming Batches Throw Exception (drill)

2024-03-05 Thread via GitHub
shfshihuafeng commented on code in PR #2878: URL: https://github.com/apache/drill/pull/2878#discussion_r1512773128 ## exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/MergeJoinBatch.java: ## @@ -297,7 +297,14 @@ public void close() {

Re: [PR] DRILL-8479: Merge Join Leak When Depleting Incoming Batches Throw Exception (drill)

2024-03-05 Thread via GitHub
shfshihuafeng commented on code in PR #2878: URL: https://github.com/apache/drill/pull/2878#discussion_r1512773128 ## exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/MergeJoinBatch.java: ## @@ -297,7 +297,14 @@ public void close() {

Re: [PR] DRILL-8479: Merge Join Leak When Depleting Incoming Batches Throw Exception (drill)

2024-03-05 Thread via GitHub
shfshihuafeng commented on code in PR #2878: URL: https://github.com/apache/drill/pull/2878#discussion_r1512773128 ## exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/MergeJoinBatch.java: ## @@ -297,7 +297,14 @@ public void close() {

Re: [PR] DRILL-8479: Merge Join Leak When Depleting Incoming Batches Throw Exception (drill)

2024-03-05 Thread via GitHub
cgivre commented on code in PR #2878: URL: https://github.com/apache/drill/pull/2878#discussion_r1512908376 ## exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/MergeJoinBatch.java: ## @@ -297,7 +297,14 @@ public void close() {

Re: [PR] DRILL-8479: Merge Join Leak When Depleting Incoming Batches Throw Exception (drill)

2024-03-05 Thread via GitHub
shfshihuafeng commented on code in PR #2878: URL: https://github.com/apache/drill/pull/2878#discussion_r1513766703 ## exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/MergeJoinBatch.java: ## @@ -297,7 +297,14 @@ public void close() {

Re: [PR] DRILL-8479: Merge Join Leak When Depleting Incoming Batches Throw Exception (drill)

2024-03-05 Thread via GitHub
shfshihuafeng commented on code in PR #2878: URL: https://github.com/apache/drill/pull/2878#discussion_r1513768876 ## exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/MergeJoinBatch.java: ## @@ -297,7 +297,14 @@ public void close() {

Re: [PR] DRILL-8479: Merge Join Leak When Depleting Incoming Batches Throw Exception (drill)

2024-03-05 Thread via GitHub
shfshihuafeng commented on code in PR #2878: URL: https://github.com/apache/drill/pull/2878#discussion_r1513773463 ## exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/MergeJoinBatch.java: ## @@ -297,7 +297,14 @@ public void close() {

Re: [PR] DRILL-8479: Merge Join Leak When Depleting Incoming Batches Throw Exception (drill)

2024-03-05 Thread via GitHub
cgivre merged PR #2878: URL: https://github.com/apache/drill/pull/2878 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@drill.apache.org