[GitHub] drill pull request: DRILL-3714: Query runs out of memory and remai...

2016-04-02 Thread adeneche
Github user adeneche commented on the pull request: https://github.com/apache/drill/pull/442#issuecomment-204802224 Just updated the fix to only fail the CoordinationQueue listeners for the closed channel. @jacques-n and @sudheeshkatkam let me know if this is acceptable. In the meanti

[GitHub] drill pull request: DRILL-3714: Query runs out of memory and remai...

2016-04-02 Thread adeneche
Github user adeneche commented on a diff in the pull request: https://github.com/apache/drill/pull/442#discussion_r58298118 --- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/RpcBus.java --- @@ -159,19 +159,15 @@ public ChannelClosedHandler(C clientConnection, Channel chann

[GitHub] drill pull request: DRILL-3714: Query runs out of memory and remai...

2016-04-02 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/442#discussion_r58296230 --- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/RpcBus.java --- @@ -159,19 +159,15 @@ public ChannelClosedHandler(C clientConnection, Channel

[GitHub] drill pull request: DRILL-3714: Query runs out of memory and remai...

2016-04-02 Thread adeneche
Github user adeneche commented on a diff in the pull request: https://github.com/apache/drill/pull/442#discussion_r58290979 --- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/RpcBus.java --- @@ -159,19 +159,15 @@ public ChannelClosedHandler(C clientConnection, Channel chann