You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by sp...@apache.org on 2016/02/25 15:01:22 UTC

[34/43] incubator-tinkerpop git commit: Fix HTTP keepalive support

Fix HTTP keepalive support

This commit[1] broke HTTP keepalive support by introducing a
double-negation on the call to Netty's isKeepAlive.

[1]: https://github.com/apache/incubator-tinkerpop/commit/c0121aae8266e9ff2c70c1517e809b5ecf1dd07a


Project: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/commit/33cd0039
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/tree/33cd0039
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/diff/33cd0039

Branch: refs/heads/TINKERPOP-1107
Commit: 33cd00390a014d6ce54c7627b00a3d1b3114bddc
Parents: 13f8005
Author: Benjamin Anderson <b...@banjiewen.net>
Authored: Tue Feb 23 17:24:15 2016 -0800
Committer: Benjamin Anderson <b...@banjiewen.net>
Committed: Wed Feb 24 10:05:20 2016 -0800

----------------------------------------------------------------------
 .../gremlin/server/handler/HttpGremlinEndpointHandler.java         | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/33cd0039/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java
----------------------------------------------------------------------
diff --git a/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java b/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java
index d37a1bc..7d2222f 100644
--- a/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java
+++ b/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/handler/HttpGremlinEndpointHandler.java
@@ -180,7 +180,7 @@ public class HttpGremlinEndpointHandler extends ChannelInboundHandlerAdapter {
             }
 
             final String origin = req.headers().get(ORIGIN);
-            final boolean keepAlive = !isKeepAlive(req);
+            final boolean keepAlive = isKeepAlive(req);
 
             // not using the req any where below here - assume it is safe to release at this point.
             ReferenceCountUtil.release(msg);