You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tinkerpop.apache.org by okram <gi...@git.apache.org> on 2016/09/13 15:07:34 UTC

[GitHub] tinkerpop issue #404: TINKERPOP-1280 VertexPrograms should declare traverser...

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.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---