You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by ok...@apache.org on 2015/04/30 20:56:45 UTC
incubator-tinkerpop git commit: minor fix ups to SideEffectCapStep.
Repository: incubator-tinkerpop
Updated Branches:
refs/heads/master d36bc4280 -> 4326a40a7
minor fix ups to SideEffectCapStep.
Project: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/commit/4326a40a
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/tree/4326a40a
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/diff/4326a40a
Branch: refs/heads/master
Commit: 4326a40a7c0dbd4bab7b8ce0aa3adbcdb1d82dbc
Parents: d36bc42
Author: Marko A. Rodriguez <ok...@gmail.com>
Authored: Thu Apr 30 12:56:42 2015 -0600
Committer: Marko A. Rodriguez <ok...@gmail.com>
Committed: Thu Apr 30 12:56:42 2015 -0600
----------------------------------------------------------------------
.../traversal/step/sideEffect/SideEffectCapStep.java | 14 +++++---------
1 file changed, 5 insertions(+), 9 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/4326a40a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SideEffectCapStep.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SideEffectCapStep.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SideEffectCapStep.java
index 830d01f..52d0f87 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SideEffectCapStep.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SideEffectCapStep.java
@@ -24,8 +24,7 @@ import org.apache.tinkerpop.gremlin.process.traversal.traverser.TraverserRequire
import org.apache.tinkerpop.gremlin.process.traversal.util.TraversalHelper;
import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.EnumSet;
+import java.util.Collections;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
@@ -36,18 +35,15 @@ import java.util.Set;
*/
public final class SideEffectCapStep<S, E> extends SupplyingBarrierStep<S, E> {
- private static final Set<TraverserRequirement> REQUIREMENTS = EnumSet.of(
- TraverserRequirement.SIDE_EFFECTS,
- TraverserRequirement.OBJECT
- );
-
private List<String> sideEffectKeys;
public SideEffectCapStep(final Traversal.Admin traversal, final String sideEffectKey, final String... sideEffectKeys) {
super(traversal);
this.sideEffectKeys = new ArrayList<>(1 + sideEffectKeys.length);
this.sideEffectKeys.add(sideEffectKey);
- this.sideEffectKeys.addAll(Arrays.asList(sideEffectKeys));
+ for (final String key : sideEffectKeys) {
+ this.sideEffectKeys.add(key);
+ }
}
@Override
@@ -61,7 +57,7 @@ public final class SideEffectCapStep<S, E> extends SupplyingBarrierStep<S, E> {
@Override
public Set<TraverserRequirement> getRequirements() {
- return REQUIREMENTS;
+ return Collections.singleton(TraverserRequirement.SIDE_EFFECTS);
}
@Override