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/09 21:18:00 UTC
incubator-tinkerpop git commit: minor tweak to TraverserExecutor.
Repository: incubator-tinkerpop
Updated Branches:
refs/heads/master 2e9d34644 -> ce33a057e
minor tweak to TraverserExecutor.
Project: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/commit/ce33a057
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/tree/ce33a057
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/diff/ce33a057
Branch: refs/heads/master
Commit: ce33a057eb8623b909ac1c4da24bef4ddbb04f8a
Parents: 2e9d346
Author: Marko A. Rodriguez <ok...@gmail.com>
Authored: Thu Apr 9 13:18:15 2015 -0600
Committer: Marko A. Rodriguez <ok...@gmail.com>
Committed: Thu Apr 9 13:18:15 2015 -0600
----------------------------------------------------------------------
.../gremlin/process/computer/traversal/TraverserExecutor.java | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/ce33a057/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/computer/traversal/TraverserExecutor.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/computer/traversal/TraverserExecutor.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/computer/traversal/TraverserExecutor.java
index d927d74..79175b1 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/computer/traversal/TraverserExecutor.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/computer/traversal/TraverserExecutor.java
@@ -30,6 +30,7 @@ import org.apache.tinkerpop.gremlin.structure.Element;
import org.apache.tinkerpop.gremlin.structure.Property;
import org.apache.tinkerpop.gremlin.structure.Vertex;
import org.apache.tinkerpop.gremlin.structure.util.detached.DetachedElement;
+import org.apache.tinkerpop.gremlin.structure.util.detached.DetachedProperty;
import java.util.concurrent.atomic.AtomicBoolean;
@@ -62,7 +63,9 @@ public final class TraverserExecutor {
if (traverser.get() instanceof Element || traverser.get() instanceof Property) { // GRAPH OBJECT
// if the element is remote, then message, else store it locally for re-processing
final Vertex hostingVertex = TraverserExecutor.getHostingVertex(traverser.get());
- if (!vertex.equals(hostingVertex) || traverser.get() instanceof DetachedElement) { // TODO: why is the DetachedElement instanceof needed?
+ if (!vertex.equals(hostingVertex)
+ || traverser.get() instanceof DetachedElement
+ || traverser.get() instanceof DetachedProperty) { // necessary for path access (but why are these not ReferenceXXX?)
voteToHalt.set(false);
traverser.detach();
messenger.sendMessage(MessageScope.Global.of(hostingVertex), new TraverserSet<>(traverser));