replaced TraversalRing's setTraversal() with replaceTraversal()
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/84df8740 Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/84df8740 Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/84df8740 Branch: refs/heads/TINKERPOP-1682 Commit: 84df874049c128cc4d899b101370fb598024aa82 Parents: 4dcd6b2 Author: Daniel Kuppitz <daniel_kupp...@hotmail.com> Authored: Fri Jun 23 16:05:30 2017 +0200 Committer: Daniel Kuppitz <daniel_kupp...@hotmail.com> Committed: Mon Mar 19 12:28:38 2018 -0700 ---------------------------------------------------------------------- .../process/traversal/step/filter/PathFilterStep.java | 11 +++-------- .../traversal/step/filter/WherePredicateStep.java | 11 +++-------- .../gremlin/process/traversal/step/map/PathStep.java | 11 +++-------- .../gremlin/process/traversal/step/map/ProjectStep.java | 11 +++-------- .../gremlin/process/traversal/step/map/SelectStep.java | 11 +++-------- .../gremlin/process/traversal/step/map/TreeStep.java | 11 +++-------- .../traversal/step/sideEffect/TreeSideEffectStep.java | 11 +++-------- .../gremlin/process/traversal/util/TraversalRing.java | 10 ++++++++-- 8 files changed, 29 insertions(+), 58 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/84df8740/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/PathFilterStep.java ---------------------------------------------------------------------- diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/PathFilterStep.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/PathFilterStep.java index 80355ea..6b77a36 100644 --- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/PathFilterStep.java +++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/PathFilterStep.java @@ -110,14 +110,9 @@ public final class PathFilterStep<S> extends FilterStep<S> implements FromToModu @Override public void replaceLocalChild(final Traversal.Admin<?, ?> oldTraversal, final Traversal.Admin<?, ?> newTraversal) { - int i = 0; - for (final Traversal.Admin<?, ?> traversal : this.traversalRing.getTraversals()) { - if (null != traversal && traversal.equals(oldTraversal)) { - this.traversalRing.setTraversal(i, this.integrateChild(newTraversal)); - break; - } - i++; - } + this.traversalRing.replaceTraversal( + (Traversal.Admin<Object, Object>) oldTraversal, + (Traversal.Admin<Object, Object>) newTraversal); } @Override http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/84df8740/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/WherePredicateStep.java ---------------------------------------------------------------------- diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/WherePredicateStep.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/WherePredicateStep.java index b9cf110..59ad435 100644 --- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/WherePredicateStep.java +++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/WherePredicateStep.java @@ -166,13 +166,8 @@ public final class WherePredicateStep<S> extends FilterStep<S> implements Scopin @Override public void replaceLocalChild(final Traversal.Admin<?, ?> oldTraversal, final Traversal.Admin<?, ?> newTraversal) { - int i = 0; - for (final Traversal.Admin<?, ?> traversal : this.traversalRing.getTraversals()) { - if (null != traversal && traversal.equals(oldTraversal)) { - this.traversalRing.setTraversal(i, this.integrateChild(newTraversal)); - break; - } - i++; - } + this.traversalRing.replaceTraversal( + (Traversal.Admin) oldTraversal, + (Traversal.Admin) newTraversal); } } http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/84df8740/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/PathStep.java ---------------------------------------------------------------------- diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/PathStep.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/PathStep.java index 80642c7..111105f 100644 --- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/PathStep.java +++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/PathStep.java @@ -100,14 +100,9 @@ public final class PathStep<S> extends MapStep<S, Path> implements TraversalPare @Override public void replaceLocalChild(final Traversal.Admin<?, ?> oldTraversal, final Traversal.Admin<?, ?> newTraversal) { - int i = 0; - for (final Traversal.Admin<?, ?> traversal : this.traversalRing.getTraversals()) { - if (null != traversal && traversal.equals(oldTraversal)) { - this.traversalRing.setTraversal(i, this.integrateChild(newTraversal)); - break; - } - i++; - } + this.traversalRing.replaceTraversal( + (Traversal.Admin<Object, Object>) oldTraversal, + (Traversal.Admin<Object, Object>) newTraversal); } @Override http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/84df8740/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/ProjectStep.java ---------------------------------------------------------------------- diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/ProjectStep.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/ProjectStep.java index 380a64a..f31d587 100644 --- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/ProjectStep.java +++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/ProjectStep.java @@ -99,14 +99,9 @@ public final class ProjectStep<S, E> extends MapStep<S, Map<String, E>> implemen @Override public void replaceLocalChild(final Traversal.Admin<?, ?> oldTraversal, final Traversal.Admin<?, ?> newTraversal) { - int i = 0; - for (final Traversal.Admin<?, ?> traversal : this.traversalRing.getTraversals()) { - if (null != traversal && traversal.equals(oldTraversal)) { - this.traversalRing.setTraversal(i, this.integrateChild(newTraversal)); - break; - } - i++; - } + this.traversalRing.replaceTraversal( + (Traversal.Admin<S, E>) oldTraversal, + (Traversal.Admin<S, E>) newTraversal); } public List<String> getProjectKeys() { http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/84df8740/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/SelectStep.java ---------------------------------------------------------------------- diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/SelectStep.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/SelectStep.java index c588862..47d0261 100644 --- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/SelectStep.java +++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/SelectStep.java @@ -119,14 +119,9 @@ public final class SelectStep<S, E> extends MapStep<S, Map<String, E>> implement @Override public void replaceLocalChild(final Traversal.Admin<?, ?> oldTraversal, final Traversal.Admin<?, ?> newTraversal) { - int i = 0; - for (final Traversal.Admin<?, ?> traversal : this.traversalRing.getTraversals()) { - if (null != traversal && traversal.equals(oldTraversal)) { - this.traversalRing.setTraversal(i, this.integrateChild(newTraversal)); - break; - } - i++; - } + this.traversalRing.replaceTraversal( + (Traversal.Admin<Object, E>) oldTraversal, + (Traversal.Admin<Object, E>) newTraversal); } @Override http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/84df8740/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/TreeStep.java ---------------------------------------------------------------------- diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/TreeStep.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/TreeStep.java index a6c65a7..1904859 100644 --- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/TreeStep.java +++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/TreeStep.java @@ -65,14 +65,9 @@ public final class TreeStep<S> extends ReducingBarrierStep<S, Tree> implements T @Override public void replaceLocalChild(final Traversal.Admin<?, ?> oldTraversal, final Traversal.Admin<?, ?> newTraversal) { - int i = 0; - for (final Traversal.Admin<?, ?> traversal : this.traversalRing.getTraversals()) { - if (null != traversal && traversal.equals(oldTraversal)) { - this.traversalRing.setTraversal(i, this.integrateChild(newTraversal)); - break; - } - i++; - } + this.traversalRing.replaceTraversal( + (Traversal.Admin<Object, Object>) oldTraversal, + (Traversal.Admin<Object, Object>) newTraversal); } @Override http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/84df8740/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/TreeSideEffectStep.java ---------------------------------------------------------------------- diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/TreeSideEffectStep.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/TreeSideEffectStep.java index d07ebee..c3f0fbd 100644 --- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/TreeSideEffectStep.java +++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/TreeSideEffectStep.java @@ -120,14 +120,9 @@ public final class TreeSideEffectStep<S> extends SideEffectStep<S> implements Si @Override public void replaceLocalChild(final Traversal.Admin<?, ?> oldTraversal, final Traversal.Admin<?, ?> newTraversal) { - int i = 0; - for (final Traversal.Admin<?, ?> traversal : this.traversalRing.getTraversals()) { - if (null != traversal && traversal.equals(oldTraversal)) { - this.traversalRing.setTraversal(i, this.integrateChild(newTraversal)); - break; - } - i++; - } + this.traversalRing.replaceTraversal( + (Traversal.Admin<Object, Object>) oldTraversal, + (Traversal.Admin<Object, Object>) newTraversal); } @Override http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/84df8740/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalRing.java ---------------------------------------------------------------------- diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalRing.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalRing.java index a3d6972..bbdf701 100644 --- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalRing.java +++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalRing.java @@ -24,6 +24,7 @@ import java.io.Serializable; import java.util.ArrayList; import java.util.Collections; import java.util.List; +import java.util.Objects; /** * @author Marko A. Rodriguez (http://markorodriguez.com) @@ -62,8 +63,13 @@ public final class TraversalRing<A, B> implements Serializable, Cloneable { this.traversals.add(traversal); } - public void setTraversal(final int index, final Traversal.Admin<A, B> traversal) { - this.traversals.set(index, traversal); + public void replaceTraversal(final Traversal.Admin<A, B> oldTraversal, final Traversal.Admin<A, B> newTraversal) { + for (int i = 0, j = this.traversals.size(); i < j; i++) { + if (Objects.equals(oldTraversal, this.traversals.get(i))) { + this.traversals.set(i, newTraversal); + break; + } + } } public List<Traversal.Admin<A, B>> getTraversals() {