Github user dimas-b commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/899#discussion_r206594732
--- Diff:
gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/AbstractOpProcessor.java
---
@@ -72,8 +73,19 @@ protected
Github user dimas-b commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/899#discussion_r206581828
--- Diff:
gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/AbstractOpProcessor.java
---
@@ -72,8 +73,19 @@ protected
Github user dimas-b commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/899#discussion_r206559210
--- Diff:
gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/ResponseHandlerContext.java
---
@@ -0,0 +1,85 @@
+/*
+ * Licensed
Github user dimas-b commented on the issue:
https://github.com/apache/tinkerpop/pull/899
I see a test failure related to log message interception in the automated
build:
```
[ERROR] shouldAllowAtMostOneFinalResponse[writeAndFlush(ResponseStatusCode,
Object
Github user dimas-b commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/899#discussion_r206549700
--- Diff:
gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/ResponseHandlerContext.java
---
@@ -0,0 +1,85 @@
+/*
+ * Licensed
Github user dimas-b commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/899#discussion_r206549368
--- Diff:
gremlin-driver/src/test/java/org/apache/tinkerpop/gremlin/driver/message/ResponseStatusCodeTest.java
---
@@ -0,0 +1,35
Github user dimas-b commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/899#discussion_r206547429
--- Diff:
gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/AbstractOpProcessor.java
---
@@ -252,15 +264,29 @@ protected boolean
Github user dimas-b commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/899#discussion_r206542682
--- Diff:
gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/AbstractEvalOpProcessor.java
---
@@ -222,9 +223,33 @@ protected
Github user dimas-b commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/899#discussion_r206540558
--- Diff:
gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/AbstractEvalOpProcessor.java
---
@@ -222,9 +223,33 @@ protected
GitHub user dimas-b opened a pull request:
https://github.com/apache/tinkerpop/pull/899
TINKERPOP-2005: Intermittent NullPointerException in response handling
Address
[TINKERPOP-2005](https://issues.apache.org/jira/browse/TINKERPOP-2005) by
ignoring more than one "final"
Github user dimas-b closed the pull request at:
https://github.com/apache/tinkerpop/pull/896
---
GitHub user dimas-b opened a pull request:
https://github.com/apache/tinkerpop/pull/896
TINKERPOP-2005 Reject multiple final responses in AbstractEvalOpProcessor
Add isFinalResponse() getter to ResponseStatusCode
Introduce writeAndFlush methods in Context to allow tracking
12 matches
Mail list logo