[
https://issues.apache.org/jira/browse/TINKERPOP-1280?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15487459#comment-15487459
]
ASF GitHub Bot commented on TINKERPOP-1280:
-------------------------------------------
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/404
Very well done code. In other words, in full arrogance and conceit, this is
exactly how I would have done it thus, you work is perfect.
And like me, I would have skimped on the JavaDoc. Can you please add
JavaDoc to `VertexProperty.getTraverserRequirements()` before merging to master.
VOTE +1.
> VertexPrograms should declare traverser requirements
> ----------------------------------------------------
>
> Key: TINKERPOP-1280
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1280
> Project: TinkerPop
> Issue Type: Improvement
> Components: process
> Affects Versions: 3.2.0-incubating
> Reporter: Daniel Kuppitz
> Assignee: Daniel Kuppitz
> Fix For: 3.2.3
>
>
> VertexPrograms should declare their {{TraverserRequirement}} which can then
> be propagated by the {{program()}} step. Most VPs (at least the ones I have
> written so far) rely on (labeled) path information and thus will fail if the
> actual traversal doesn't use any other steps with the same traverser
> requirements. I keep running into this issue whenever I write a new VP.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
