abdullah alamoudi has posted comments on this change.

Change subject: [ASTERIXDB-2361][HYR] Memory Leak Due to Netty Close Listeners
......................................................................


Patch Set 3:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/2579/3/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/ChunkedNettyOutputStream.java
File 
hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/ChunkedNettyOutputStream.java:

PS3, Line 46: // register listener for channel closed
            :         ctx.channel().closeFuture().addListener(futureListener -> 
{
            :             synchronized (ChunkedNettyOutputStream.this) {
            :                 ChunkedNettyOutputStream.this.notifyAll();
            :             }
            :         });
we should remove this


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2579
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I40156538d62a3c06b9ccc14338c3f554921a12b8
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: release-0.9.4-pre-rc
Gerrit-Owner: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: abdullah alamoudi <bamou...@gmail.com>
Gerrit-HasComments: Yes

Reply via email to