Murtadha Hubail has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/3006
Change subject: [NO ISSUE][OTH] Do Not Close Client Connection After Failure
......................................................................
[NO ISSUE][OTH] Do Not Close Client Connection After Failure
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
- Currently, after sending some failures responses (e.g. after
servlet not found), the client connection is closed even if
the connection was supposed to be kept alive. This change
ensures that we do not close the client connection which allows
the client to submit another request using the same connection.
Change-Id: Id0fce2c860eec97f3d368ee42f25dbdfc9dc0ff9
---
M
hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServerHandler.java
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/06/3006/1
diff --git
a/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServerHandler.java
b/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServerHandler.java
index baa664a..59f6d0f 100644
---
a/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServerHandler.java
+++
b/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServerHandler.java
@@ -98,7 +98,7 @@
protected void respond(ChannelHandlerContext ctx, HttpVersion httpVersion,
HttpResponseStatus status) {
DefaultHttpResponse response = new DefaultHttpResponse(httpVersion,
status);
- ctx.writeAndFlush(response).addListener(ChannelFutureListener.CLOSE);
+ ctx.writeAndFlush(response);
}
private void submit(ChannelHandlerContext ctx, IServlet servlet,
FullHttpRequest request) throws IOException {
--
To view, visit https://asterix-gerrit.ics.uci.edu/3006
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id0fce2c860eec97f3d368ee42f25dbdfc9dc0ff9
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <[email protected]>
