[
https://issues.apache.org/jira/browse/TINKERPOP-1652?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15929956#comment-15929956
]
ASF GitHub Bot commented on TINKERPOP-1652:
-------------------------------------------
Github user twilmes commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/572#discussion_r106646840
--- Diff:
gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/optimization/PathRetractionStrategyTest.java
---
@@ -191,6 +206,9 @@ public void doTest() {
{__.V().as("a").optional(bothE().dedup().as("b")).
choose(select("b"), select("a","b"),
project("a").by(select("a"))),
"[[[a, b]], [[a, b]], [[a, b]], [[[a, b]]], [[a,
b]]]", null},
+
{__.V().as("a").out().program(labeledPathVertexProgram).select("a"), "[]",
null},
+
{__.V().as("a").out().program(pathVertexProgram).select("a"), "[]", null},
+
{__.V().as("a").out().program(emptyRequirementsVertexProgram).select("a"),
"[[]]", null}
--- End diff --
Yeah, that is confusing. It is correctly asserting with that `[]` but it
is non-obvious. I replaced those with a constant and then reworked the
traversals.
> Disable PathRetractionStrategy strategy if VertexProgramStep has
> LABELLED_PATH requirement
> ------------------------------------------------------------------------------------------
>
> Key: TINKERPOP-1652
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1652
> Project: TinkerPop
> Issue Type: Bug
> Reporter: Ted Wilmes
> Assignee: Ted Wilmes
>
> [~dkuppitz] notified me that there is an issue where
> {{PathRetractionStrategy}} is incorrectly dropping path elements when a
> {{VertexProgramStep}} is present that has a {{LABELLED_PATH}} requirement.
> We already handle the case where path is required but this issue will also
> disable the strategy if a vertex program is present that requires
> {{LABELLED_PATH}}.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
