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 2018/07/12 12:19:06 UTC

tinkerpop git commit: TINKERPOP-1996 none() doesn't need to be removed in HadoopIoStrategy

Repository: tinkerpop
Updated Branches:
  refs/heads/TINKERPOP-1996 435ebdc1f -> caa2d1d86


TINKERPOP-1996 none() doesn't need to be removed in HadoopIoStrategy

Not sure why this was there in the first place. Removing it not allows Hadoop integration tests to pass, but seems to have no real effect on existing operations.


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

Branch: refs/heads/TINKERPOP-1996
Commit: caa2d1d867248c38f6407cc0063af4864e11d163
Parents: 435ebdc
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Thu Jul 12 08:17:56 2018 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Thu Jul 12 08:17:56 2018 -0400

----------------------------------------------------------------------
 .../process/computer/traversal/strategy/HadoopIoStrategy.java     | 3 ---
 1 file changed, 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/caa2d1d8/hadoop-gremlin/src/main/java/org/apache/tinkerpop/gremlin/hadoop/process/computer/traversal/strategy/HadoopIoStrategy.java
----------------------------------------------------------------------
diff --git a/hadoop-gremlin/src/main/java/org/apache/tinkerpop/gremlin/hadoop/process/computer/traversal/strategy/HadoopIoStrategy.java b/hadoop-gremlin/src/main/java/org/apache/tinkerpop/gremlin/hadoop/process/computer/traversal/strategy/HadoopIoStrategy.java
index 38d5a7f..89ee04b 100644
--- a/hadoop-gremlin/src/main/java/org/apache/tinkerpop/gremlin/hadoop/process/computer/traversal/strategy/HadoopIoStrategy.java
+++ b/hadoop-gremlin/src/main/java/org/apache/tinkerpop/gremlin/hadoop/process/computer/traversal/strategy/HadoopIoStrategy.java
@@ -60,9 +60,6 @@ public final class HadoopIoStrategy extends AbstractTraversalStrategy<TraversalS
 
             TraversalHelper.replaceStep((Step) readWriting, hadoopIoStep, traversal);
         }
-
-        if (traversal.getEndStep() instanceof NoneStep)
-            traversal.removeStep(1);
     }
 
     public static HadoopIoStrategy instance() {