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/02/17 22:41:25 UTC
[1/5] incubator-tinkerpop git commit: Fixed a bug in `select()`,
when it assumed that every map in a result is the result of a
`match()` step.
Repository: incubator-tinkerpop
Updated Branches:
refs/heads/master 0e7a612a8 -> 6f2474fc3
Fixed a bug in `select()`, when it assumed that every map in a result is the result of a `match()` step.
Project: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/commit/22e2fec0
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/tree/22e2fec0
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/diff/22e2fec0
Branch: refs/heads/master
Commit: 22e2fec03991db194412eca1f645f3eaad89bd85
Parents: 2ba09b4
Author: Daniel Kuppitz <da...@hotmail.com>
Authored: Tue Feb 17 17:33:16 2015 +0100
Committer: Daniel Kuppitz <da...@hotmail.com>
Committed: Tue Feb 17 17:33:16 2015 +0100
----------------------------------------------------------------------
.../graph/traversal/step/map/SelectStep.java | 12 +++------
.../tinkergraph/structure/TinkerGraphTest.java | 28 +++++++-------------
2 files changed, 12 insertions(+), 28 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/22e2fec0/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectStep.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectStep.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectStep.java
index ee1d840..92b0f96 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectStep.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectStep.java
@@ -30,11 +30,7 @@ import org.apache.tinkerpop.gremlin.process.traversal.util.TraversalRing;
import org.apache.tinkerpop.gremlin.process.traversal.util.TraversalUtil;
import org.apache.tinkerpop.gremlin.process.traverser.TraverserRequirement;
-import java.util.Arrays;
-import java.util.LinkedHashMap;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
+import java.util.*;
/**
* @author Marko A. Rodriguez (http://markorodriguez.com)
@@ -43,13 +39,11 @@ public final class SelectStep<S, E> extends MapStep<S, Map<String, E>> implement
protected TraversalRing<Object, Object> traversalRing = new TraversalRing<>();
private final List<String> selectLabels;
- private final boolean wasEmpty;
private boolean requiresPaths = false;
public SelectStep(final Traversal.Admin traversal, final String... selectLabels) {
super(traversal);
- this.wasEmpty = selectLabels.length == 0;
- this.selectLabels = this.wasEmpty ? TraversalHelper.getLabelsUpTo(this, this.traversal) : Arrays.asList(selectLabels);
+ this.selectLabels = selectLabels.length == 0 ? TraversalHelper.getLabelsUpTo(this, this.traversal) : Arrays.asList(selectLabels);
}
@Override
@@ -66,7 +60,7 @@ public final class SelectStep<S, E> extends MapStep<S, Map<String, E>> implement
////// PROCESS MAP BINDINGS
if (start instanceof Map) {
- if (this.wasEmpty)
+ if (this.selectLabels.isEmpty())
((Map<String, Object>) start).forEach((k, v) -> bindings.put(k, (E) TraversalUtil.apply(v, this.traversalRing.next())));
else
this.selectLabels.forEach(label -> {
http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/22e2fec0/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
----------------------------------------------------------------------
diff --git a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
index 22c827e..f57c468 100644
--- a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
+++ b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
@@ -23,6 +23,7 @@ import org.apache.tinkerpop.gremlin.AbstractGremlinTest;
import org.apache.tinkerpop.gremlin.process.Scope;
import org.apache.tinkerpop.gremlin.process.T;
import org.apache.tinkerpop.gremlin.process.Traversal;
+import org.apache.tinkerpop.gremlin.process.graph.traversal.step.map.match.Bindings;
import org.apache.tinkerpop.gremlin.process.traversal.engine.ComputerTraversalEngine;
import org.apache.tinkerpop.gremlin.structure.*;
import org.apache.tinkerpop.gremlin.structure.io.GraphReader;
@@ -40,6 +41,7 @@ import java.io.*;
import java.util.Arrays;
import java.util.List;
import java.util.Set;
+import java.util.function.Function;
import java.util.function.Supplier;
import static org.apache.tinkerpop.gremlin.process.graph.traversal.__.*;
@@ -179,29 +181,17 @@ public class TinkerGraphTest {
@Test
@Ignore
public void testPlayDK() throws Exception {
-
Graph g = TinkerFactory.createModern();
- Traversal t = g.V().hasLabel("person").as("p").local(out("created").values("name").fold().limit(Scope.local, 1));
- t.forEachRemaining(System.out::println);
- System.out.println("--");
-
- t = g.V().hasLabel("person").group().by("name").by(out("created").values("name").fold()).cap().limit(Scope.local, 2);
- t.forEachRemaining(System.out::println);
- System.out.println("--");
-
- t = g.V().hasLabel("person").as("p").local(out("created").values("name").fold().sample(Scope.local, 1));
+ Traversal t = g.V().hasLabel("person").as("person").local(bothE().label().groupCount().cap()).as("relations").select().by("name").by();
t.forEachRemaining(System.out::println);
System.out.println("--");
- t = g.V().hasLabel("person").group().by("name").by(out("created").values("name").fold()).cap().sample(Scope.local, 2);
- t.forEachRemaining(System.out::println);
- System.out.println("--");
-
- t = g.V().group().by(T.label).by(bothE().values("weight").fold()).by(sample(Scope.local, 5)).cap();
- t.forEachRemaining(System.out::println);
-
- System.out.println("--");
- t = g.V().group().by(T.label).by(bothE().values("weight").fold()).by(order(Scope.local).by(Order.keyDecr)).cap();
+ t = g.V().match("a",
+ as("a").out("knows").as("b"),
+ as("b").out("created").has("name", "lop"),
+ as("b").match("a1",
+ as("a1").out("created").as("b1"),
+ as("b1").in("created").as("c1")).select("c1").as("c")).<String>select().by("name");
t.forEachRemaining(System.out::println);
}
[5/5] incubator-tinkerpop git commit: Merge branch 'select-bug' of
https://github.com/dkuppitz/incubator-tinkerpop
Posted by ok...@apache.org.
Merge branch 'select-bug' of https://github.com/dkuppitz/incubator-tinkerpop
Project: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/commit/6f2474fc
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/tree/6f2474fc
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/diff/6f2474fc
Branch: refs/heads/master
Commit: 6f2474fc38ac664147390843f5ed500ac8abec04
Parents: 4aed87b 9ed92bf
Author: Marko A. Rodriguez <ok...@gmail.com>
Authored: Tue Feb 17 14:41:23 2015 -0700
Committer: Marko A. Rodriguez <ok...@gmail.com>
Committed: Tue Feb 17 14:41:23 2015 -0700
----------------------------------------------------------------------
.../graph/traversal/step/map/SelectStep.java | 12 +--
.../traversal/step/map/GroovySelectTest.groovy | 26 ++++++
.../graph/traversal/step/map/SelectTest.java | 95 ++++++++++++++++++--
.../tinkergraph/structure/TinkerGraphTest.java | 27 ++----
4 files changed, 126 insertions(+), 34 deletions(-)
----------------------------------------------------------------------
[2/5] incubator-tinkerpop git commit: Added more `select()` tests
that verify that the latest changes on `SelectStep` work as expected.
Posted by ok...@apache.org.
Added more `select()` tests that verify that the latest changes on `SelectStep` work as expected.
Project: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/commit/b61e0ee9
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/tree/b61e0ee9
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/diff/b61e0ee9
Branch: refs/heads/master
Commit: b61e0ee9c7f9ca4831b4b286bbfbde33c9d5b0e0
Parents: 22e2fec
Author: Daniel Kuppitz <da...@hotmail.com>
Authored: Tue Feb 17 22:24:35 2015 +0100
Committer: Daniel Kuppitz <da...@hotmail.com>
Committed: Tue Feb 17 22:24:35 2015 +0100
----------------------------------------------------------------------
.../traversal/step/map/GroovySelectTest.groovy | 24 +++++
.../graph/traversal/step/map/SelectTest.java | 94 ++++++++++++++++++--
.../tinkergraph/structure/TinkerGraphTest.java | 3 -
3 files changed, 113 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/b61e0ee9/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/GroovySelectTest.groovy
----------------------------------------------------------------------
diff --git a/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/GroovySelectTest.groovy b/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/GroovySelectTest.groovy
index 319bca7..de0e5cd 100644
--- a/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/GroovySelectTest.groovy
+++ b/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/GroovySelectTest.groovy
@@ -20,6 +20,7 @@ package org.apache.tinkerpop.gremlin.process.graph.traversal.step.map
import org.apache.tinkerpop.gremlin.process.ComputerTestHelper
import org.apache.tinkerpop.gremlin.process.Traversal
+import org.apache.tinkerpop.gremlin.process.graph.traversal.__
import org.apache.tinkerpop.gremlin.process.traversal.engine.StandardTraversalEngine
import org.apache.tinkerpop.gremlin.structure.Order
import org.apache.tinkerpop.gremlin.structure.Vertex
@@ -79,6 +80,16 @@ public abstract class GroovySelectTest {
public Traversal<Vertex, Map<String, Object>> get_g_V_hasXname_isXmarkoXX_asXaX_select() {
return g.V.has(values('name').is('marko')).as('a').select
}
+
+ @Override
+ public Traversal<Vertex, Map<String, Object>> get_g_V_label_groupCount_cap_asXxX_select() {
+ return g.V().label().groupCount().cap().as('x').select()
+ }
+
+ @Override
+ public Traversal<Vertex, Map<String, Object>> get_g_V_hasLabelXpersonX_asXpersonX_localXbothE_label_groupCount_capX_asXrelationsX_select_byXnameX_by() {
+ return g.V().hasLabel('person').as('person').local(__.bothE().label().groupCount().cap()).as('relations').select().by('name').by()
+ }
}
public static class ComputerTest extends SelectTest {
@@ -138,5 +149,18 @@ public abstract class GroovySelectTest {
public Traversal<Vertex, Map<String, Object>> get_g_V_hasXname_isXmarkoXX_asXaX_select() {
ComputerTestHelper.compute("g.V.has(values('name').is('marko')).as('a').select", g)
}
+
+ @Override
+ public Traversal<Vertex, Map<String, Object>> get_g_V_label_groupCount_cap_asXxX_select() {
+ ComputerTestHelper.compute("g.V().label().groupCount().cap().as('x').select()", g)
+ }
+
+ @Override
+ Traversal<Vertex, Map<String, Object>> get_g_V_hasLabelXpersonX_asXpersonX_localXbothE_label_groupCount_capX_asXrelationsX_select_byXnameX_by() {
+ g.engine(StandardTraversalEngine.standard);
+ // TODO: should work in computer mode, but throws ClassCastException
+ g.V().hasLabel('person').as('person').local(__.bothE().label().groupCount().cap()).as('relations').select().by('name').by()
+ //ComputerTestHelper.compute("g.V().hasLabel('person').as('person').local(__.bothE().label().groupCount().cap()).as('relations').select().by('name').by()", g)
+ }
}
}
http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/b61e0ee9/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectTest.java
----------------------------------------------------------------------
diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectTest.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectTest.java
index 813c789..d5189e7 100644
--- a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectTest.java
+++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectTest.java
@@ -21,20 +21,19 @@ package org.apache.tinkerpop.gremlin.process.graph.traversal.step.map;
import org.apache.tinkerpop.gremlin.LoadGraphWith;
import org.apache.tinkerpop.gremlin.process.AbstractGremlinProcessTest;
import org.apache.tinkerpop.gremlin.process.Traversal;
+import org.apache.tinkerpop.gremlin.process.graph.traversal.__;
import org.apache.tinkerpop.gremlin.process.traversal.engine.StandardTraversalEngine;
import org.apache.tinkerpop.gremlin.structure.Order;
import org.apache.tinkerpop.gremlin.structure.Vertex;
+import org.junit.Ignore;
import org.junit.Test;
-import java.util.Arrays;
-import java.util.List;
-import java.util.Map;
+import java.util.*;
import static org.apache.tinkerpop.gremlin.LoadGraphWith.GraphData.CREW;
import static org.apache.tinkerpop.gremlin.LoadGraphWith.GraphData.MODERN;
import static org.apache.tinkerpop.gremlin.process.graph.traversal.__.values;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.*;
/**
* @author Marko A. Rodriguez (http://markorodriguez.com)
@@ -60,6 +59,10 @@ public abstract class SelectTest extends AbstractGremlinProcessTest {
public abstract Traversal<Vertex, Map<String, Object>> get_g_V_hasXname_isXmarkoXX_asXaX_select();
+ public abstract Traversal<Vertex, Map<String, Object>> get_g_V_label_groupCount_cap_asXxX_select();
+
+ public abstract Traversal<Vertex, Map<String, Object>> get_g_V_hasLabelXpersonX_asXpersonX_localXbothE_label_groupCount_capX_asXrelationsX_select_byXnameX_by();
+
@Test
@LoadGraphWith(MODERN)
public void g_VX1X_asXaX_outXknowsX_asXbX_select() {
@@ -176,6 +179,66 @@ public abstract class SelectTest extends AbstractGremlinProcessTest {
checkResults(expected, traversal);
}
+ @Test
+ @LoadGraphWith(MODERN)
+ public void g_V_label_groupCount_cap_asXxX_select() {
+ final Traversal<Vertex, Map<String, Object>> traversal = get_g_V_label_groupCount_cap_asXxX_select();
+ printTraversalForm(traversal);
+ assertTrue(traversal.hasNext());
+ final Map<String, Object> map1 = traversal.next();
+ assertEquals(1, map1.size());
+ assertTrue(map1.containsKey("x"));
+ final Map<String, Long> map2 = (Map<String, Long>) map1.get("x");
+ assertEquals(2, map2.size());
+ assertTrue(map2.containsKey("person"));
+ assertTrue(map2.containsKey("software"));
+ assertEquals(2, map2.get("software").longValue());
+ assertEquals(4, map2.get("person").longValue());
+ assertFalse(traversal.hasNext());
+ }
+
+ @Test
+ @Ignore
+ @LoadGraphWith(MODERN)
+ public void g_V_hasLabelXpersonX_asXpersonX_localXbothE_label_groupCount_capX_asXrelationsX_select_byXnameX_by() {
+ final Traversal<Vertex, Map<String, Object>> traversal = get_g_V_hasLabelXpersonX_asXpersonX_localXbothE_label_groupCount_capX_asXrelationsX_select_byXnameX_by();
+ printTraversalForm(traversal);
+ final Set<String> persons = new HashSet<>();
+ for (int i = 0; i < 4; i++) {
+ assertTrue(traversal.hasNext());
+ final Map<String, Object> map = traversal.next();
+ assertEquals(2, map.size());
+ assertTrue(map.containsKey("person"));
+ assertTrue(map.containsKey("relations"));
+ assertTrue(persons.add((String) map.get("person")));
+ final Map<String, Long> relations = (Map<String, Long>) map.get("relations");
+ switch ((String) map.get("person")) {
+ case "marko":
+ assertEquals(2, relations.size());
+ assertEquals(1, relations.get("created").longValue());
+ assertEquals(2, relations.get("knows").longValue());
+ break;
+ case "vadas":
+ assertEquals(1, relations.size());
+ assertEquals(1, relations.get("knows").longValue());
+ break;
+ case "josh":
+ assertEquals(2, relations.size());
+ assertEquals(2, relations.get("created").longValue());
+ assertEquals(1, relations.get("knows").longValue());
+ break;
+ case "peter":
+ assertEquals(1, relations.size());
+ assertEquals(1, relations.get("created").longValue());
+ break;
+ default:
+ assertTrue(false);
+ break;
+ }
+ }
+ assertFalse(traversal.hasNext());
+ assertEquals(4, persons.size());
+ }
public static class StandardTest extends SelectTest {
public StandardTest() {
@@ -226,6 +289,16 @@ public abstract class SelectTest extends AbstractGremlinProcessTest {
public Traversal<Vertex, Map<String, Object>> get_g_V_hasXname_isXmarkoXX_asXaX_select() {
return g.V().has(values("name").is("marko")).as("a").select();
}
+
+ @Override
+ public Traversal<Vertex, Map<String, Object>> get_g_V_label_groupCount_cap_asXxX_select() {
+ return g.V().label().groupCount().cap().as("x").select();
+ }
+
+ @Override
+ public Traversal<Vertex, Map<String, Object>> get_g_V_hasLabelXpersonX_asXpersonX_localXbothE_label_groupCount_capX_asXrelationsX_select_byXnameX_by() {
+ return g.V().hasLabel("person").as("person").local(__.bothE().label().groupCount().cap()).as("relations").select().by("name").by();
+ }
}
public static class ComputerTest extends SelectTest {
@@ -290,5 +363,16 @@ public abstract class SelectTest extends AbstractGremlinProcessTest {
public Traversal<Vertex, Map<String, Object>> get_g_V_hasXname_isXmarkoXX_asXaX_select() {
return g.V().has(values("name").is("marko")).as("a").select();
}
+
+ @Override
+ public Traversal<Vertex, Map<String, Object>> get_g_V_label_groupCount_cap_asXxX_select() {
+ return g.V().label().groupCount().cap().as("x").select();
+ }
+
+ @Override
+ public Traversal<Vertex, Map<String, Object>> get_g_V_hasLabelXpersonX_asXpersonX_localXbothE_label_groupCount_capX_asXrelationsX_select_byXnameX_by() {
+ g.engine(StandardTraversalEngine.standard); // TODO
+ return g.V().hasLabel("person").as("person").local(__.bothE().label().groupCount().cap()).as("relations").select().by("name").by();
+ }
}
}
http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/b61e0ee9/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
----------------------------------------------------------------------
diff --git a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
index f57c468..4036089 100644
--- a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
+++ b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
@@ -20,10 +20,8 @@ package org.apache.tinkerpop.gremlin.tinkergraph.structure;
import org.apache.commons.io.FileUtils;
import org.apache.tinkerpop.gremlin.AbstractGremlinTest;
-import org.apache.tinkerpop.gremlin.process.Scope;
import org.apache.tinkerpop.gremlin.process.T;
import org.apache.tinkerpop.gremlin.process.Traversal;
-import org.apache.tinkerpop.gremlin.process.graph.traversal.step.map.match.Bindings;
import org.apache.tinkerpop.gremlin.process.traversal.engine.ComputerTraversalEngine;
import org.apache.tinkerpop.gremlin.structure.*;
import org.apache.tinkerpop.gremlin.structure.io.GraphReader;
@@ -41,7 +39,6 @@ import java.io.*;
import java.util.Arrays;
import java.util.List;
import java.util.Set;
-import java.util.function.Function;
import java.util.function.Supplier;
import static org.apache.tinkerpop.gremlin.process.graph.traversal.__.*;
[3/5] incubator-tinkerpop git commit: Excluded tests with
mid-traversal `cap()` in `ComputerTestSuite`.
Posted by ok...@apache.org.
Excluded tests with mid-traversal `cap()` in `ComputerTestSuite`.
Project: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/commit/9ed92bfa
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/tree/9ed92bfa
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/diff/9ed92bfa
Branch: refs/heads/master
Commit: 9ed92bfa22c5e6b5d9b7f15875f4988c93c9c4fc
Parents: b61e0ee
Author: Daniel Kuppitz <da...@hotmail.com>
Authored: Tue Feb 17 22:27:42 2015 +0100
Committer: Daniel Kuppitz <da...@hotmail.com>
Committed: Tue Feb 17 22:27:42 2015 +0100
----------------------------------------------------------------------
.../process/graph/traversal/step/map/GroovySelectTest.groovy | 4 +++-
.../gremlin/process/graph/traversal/step/map/SelectTest.java | 1 +
2 files changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/9ed92bfa/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/GroovySelectTest.groovy
----------------------------------------------------------------------
diff --git a/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/GroovySelectTest.groovy b/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/GroovySelectTest.groovy
index de0e5cd..6809429 100644
--- a/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/GroovySelectTest.groovy
+++ b/gremlin-groovy-test/src/main/groovy/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/GroovySelectTest.groovy
@@ -152,7 +152,9 @@ public abstract class GroovySelectTest {
@Override
public Traversal<Vertex, Map<String, Object>> get_g_V_label_groupCount_cap_asXxX_select() {
- ComputerTestHelper.compute("g.V().label().groupCount().cap().as('x').select()", g)
+ g.engine(StandardTraversalEngine.standard);
+ g.V().label().groupCount().cap().as('x').select()
+ //ComputerTestHelper.compute("g.V().label().groupCount().cap().as('x').select()", g)
}
@Override
http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/9ed92bfa/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectTest.java
----------------------------------------------------------------------
diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectTest.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectTest.java
index d5189e7..23823e8 100644
--- a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectTest.java
+++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/step/map/SelectTest.java
@@ -366,6 +366,7 @@ public abstract class SelectTest extends AbstractGremlinProcessTest {
@Override
public Traversal<Vertex, Map<String, Object>> get_g_V_label_groupCount_cap_asXxX_select() {
+ g.engine(StandardTraversalEngine.standard); // TODO
return g.V().label().groupCount().cap().as("x").select();
}
[4/5] incubator-tinkerpop git commit: __ is just a redirect for
inject().
Posted by ok...@apache.org.
__ is just a redirect for inject().
Project: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/commit/4aed87bd
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/tree/4aed87bd
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/diff/4aed87bd
Branch: refs/heads/master
Commit: 4aed87bd8749b18b48171a818f79b2a00d8f3bfe
Parents: 0e7a612
Author: Marko A. Rodriguez <ok...@gmail.com>
Authored: Tue Feb 17 14:41:18 2015 -0700
Committer: Marko A. Rodriguez <ok...@gmail.com>
Committed: Tue Feb 17 14:41:18 2015 -0700
----------------------------------------------------------------------
.../apache/tinkerpop/gremlin/process/graph/traversal/__.java | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/4aed87bd/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/__.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/__.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/__.java
index 385ef0f..732bdd1 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/__.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/graph/traversal/__.java
@@ -23,7 +23,6 @@ import org.apache.tinkerpop.gremlin.process.Scope;
import org.apache.tinkerpop.gremlin.process.T;
import org.apache.tinkerpop.gremlin.process.Traversal;
import org.apache.tinkerpop.gremlin.process.Traverser;
-import org.apache.tinkerpop.gremlin.process.graph.traversal.step.sideEffect.StartStep;
import org.apache.tinkerpop.gremlin.structure.Direction;
import org.apache.tinkerpop.gremlin.structure.Edge;
import org.apache.tinkerpop.gremlin.structure.Property;
@@ -56,10 +55,8 @@ public class __ {
return new DefaultGraphTraversal<>(new __());
}
- public static <A> GraphTraversal<A, A> __(final Object start) {
- final GraphTraversal.Admin<A, A> traversal = new DefaultGraphTraversal<>(start);
- traversal.addStep(new StartStep<>(traversal, start));
- return traversal;
+ public static <A> GraphTraversal<A, A> __(final Object... starts) {
+ return inject(starts);
}
///////////////////// MAP STEPS /////////////////////