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/10 16:44:20 UTC

tinkerpop git commit: Fixed side-effect logger messaging.

Repository: tinkerpop
Updated Branches:
  refs/heads/TINKERPOP-1278 c815ece40 -> 6efca4ad0


Fixed side-effect logger messaging.


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

Branch: refs/heads/TINKERPOP-1278
Commit: 6efca4ad0a10975e8d61042fd9da26a7d448226b
Parents: c815ece
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Wed Aug 10 12:44:05 2016 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Wed Aug 10 12:44:05 2016 -0400

----------------------------------------------------------------------
 .../gremlin/server/op/traversal/TraversalOpProcessor.java      | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/6efca4ad/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 6eed85d..9d59149 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
@@ -285,7 +285,7 @@ public class TraversalOpProcessor extends AbstractOpProcessor {
                         if (supportsTransactions && graph.tx().isOpen()) graph.tx().rollback();
                         return;
                     } catch (Exception ex) {
-                        logger.warn(String.format("Exception processing a Traversal on iteration for request [%s].", msg.getRequestId()), ex);
+                        logger.warn(String.format("Exception processing a side-effect on iteration for request [%s].", msg.getRequestId()), ex);
                         ctx.writeAndFlush(ResponseMessage.build(msg).code(ResponseStatusCode.SERVER_ERROR).statusMessage(ex.getMessage()).create());
                         if (supportsTransactions && graph.tx().isOpen()) graph.tx().rollback();
                         return;
@@ -295,7 +295,7 @@ public class TraversalOpProcessor extends AbstractOpProcessor {
                     // just close with rollback
                     if (supportsTransactions && graph.tx().isOpen()) graph.tx().rollback();
                 } catch (Exception ex) {
-                    logger.warn(String.format("Exception processing a Traversal on request [%s].", msg.getRequestId()), ex);
+                    logger.warn(String.format("Exception processing a side-effect on request [%s].", msg.getRequestId()), ex);
                     ctx.writeAndFlush(ResponseMessage.build(msg).code(ResponseStatusCode.SERVER_ERROR).statusMessage(ex.getMessage()).create());
                     if (graph.features().graph().supportsTransactions() && graph.tx().isOpen()) graph.tx().rollback();
                 } finally {
@@ -305,7 +305,7 @@ public class TraversalOpProcessor extends AbstractOpProcessor {
 
         } catch (Exception ex) {
             timerContext.stop();
-            throw new OpProcessorException("Could not iterate the Traversal instance",
+            throw new OpProcessorException("Could not iterate the side-effect instance",
                     ResponseMessage.build(msg).code(ResponseStatusCode.SERVER_ERROR).statusMessage(ex.getMessage()).create());
         }
     }