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/f78c87da Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/f78c87da Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/f78c87da Branch: refs/heads/TINKERPOP-1996 Commit: f78c87da3b7b4f98b168880070d2592a39aed2e9 Parents: 28c7fad 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 10:59:39 2018 -0400 ---------------------------------------------------------------------- .../process/computer/traversal/strategy/HadoopIoStrategy.java | 3 --- 1 file changed, 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/f78c87da/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() {