Github user weijietong closed the pull request at:
https://github.com/apache/drill/pull/925
---
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/925#discussion_r137955415
--- Diff:
exec/rpc/src/main/java/org/apache/drill/exec/rpc/RequestIdMap.java ---
@@ -54,10 +52,14 @@ void channelClosed(Throwable ex) {
Github user weijietong commented on a diff in the pull request:
https://github.com/apache/drill/pull/925#discussion_r137953813
--- Diff:
exec/rpc/src/main/java/org/apache/drill/exec/rpc/RequestIdMap.java ---
@@ -54,10 +52,14 @@ void channelClosed(Throwable ex) {
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/925#discussion_r137938116
--- Diff:
exec/rpc/src/main/java/org/apache/drill/exec/rpc/RequestIdMap.java ---
@@ -54,10 +52,14 @@ void channelClosed(Throwable ex) {
GitHub user weijietong opened a pull request:
https://github.com/apache/drill/pull/925
DRILL-5749: solve foreman and netty threads deadlock
break the nest invocation of channelClosed method to avoid nested lock
holding
You can merge this pull request into a Git repository by