[ 
https://issues.apache.org/jira/browse/TINKERPOP3-970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15023366#comment-15023366
 ] 

ASF GitHub Bot commented on TINKERPOP3-970:
-------------------------------------------

Github user twilmes commented on the pull request:

    
https://github.com/apache/incubator-tinkerpop/pull/153#issuecomment-159105525
  
    `mvn clean install` was good.  When I tried it out in the shell, I noticed 
if you still do a __.profile(), you get an NPE as opposed to a missing method 
exception. Doesn't seem like a huge deal because folks wouldn't profile 
straight off of __ anyways but is this expected behavior?


> ProfileStep should be off Traversal, not GraphTraversal
> -------------------------------------------------------
>
>                 Key: TINKERPOP3-970
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP3-970
>             Project: TinkerPop 3
>          Issue Type: Improvement
>          Components: process
>    Affects Versions: 3.1.0-incubating
>            Reporter: Marko A. Rodriguez
>            Assignee: Marko A. Rodriguez
>             Fix For: 3.1.1-incubating
>
>
> We currently have {{GraphTraversal.profile()}} but NOT 
> {{Traversal.profile()}}. This should be put at the {{Traversal}}-level as its 
> universal to all traversals much like {{explain()}}, {{toList()}}, 
> {{toSet()}}, {{toStream()}}, {{iterate()}}, {{hasNext()}}, {{next()}}, etc.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to