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/08/27 00:54:34 UTC
[19/50] [abbrv] tinkerpop git commit: Fixed error code when keys are
not found for traversalopprocessor
Fixed error code when keys are not found for traversalopprocessor
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/e71ebed6
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/e71ebed6
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/e71ebed6
Branch: refs/heads/master
Commit: e71ebed62b5ecd2a0023a79017012d5ea639a101
Parents: 192da0a
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Thu Aug 25 09:08:05 2016 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Thu Aug 25 09:08:05 2016 -0400
----------------------------------------------------------------------
.../gremlin/server/op/traversal/TraversalOpProcessor.java | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/e71ebed6/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/traversal/TraversalOpProcessor.java
----------------------------------------------------------------------
diff --git a/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/traversal/TraversalOpProcessor.java b/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/traversal/TraversalOpProcessor.java
index 632008c..b44c51b 100644
--- a/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/traversal/TraversalOpProcessor.java
+++ b/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/traversal/TraversalOpProcessor.java
@@ -279,7 +279,7 @@ public class TraversalOpProcessor extends AbstractOpProcessor {
if (null == sideEffects) {
final String errorMessage = String.format("Could not find side-effects for %s.", sideEffect.get());
logger.warn(errorMessage);
- ctx.writeAndFlush(ResponseMessage.build(msg).code(ResponseStatusCode.SERVER_ERROR_TIMEOUT).statusMessage(errorMessage).create());
+ ctx.writeAndFlush(ResponseMessage.build(msg).code(ResponseStatusCode.SERVER_ERROR).statusMessage(errorMessage).create());
onError(graph, context);
return;
}
@@ -319,8 +319,7 @@ public class TraversalOpProcessor extends AbstractOpProcessor {
final RequestMessage msg = context.getRequestMessage();
logger.debug("Traversal request {} for in thread {}", msg.getRequestId(), Thread.currentThread().getName());
- // TODO: Look to polish this up in GraphSON 2.0 when we don't have type lossiness anymore
- // right now the TraversalOpPorcessor can take a direct GraphSON representation of Bytecode or directly take
+ // right now the TraversalOpProcessor can take a direct GraphSON representation of Bytecode or directly take
// deserialized Bytecode object.
final Object bytecodeObj = msg.getArgs().get(Tokens.ARGS_GREMLIN);
final Bytecode bytecode = bytecodeObj instanceof Bytecode ? (Bytecode) bytecodeObj :