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

stephen mallette commented on TINKERPOP3-783:
---------------------------------------------

Note that this issue was identified on TINKERPOP3-770 as a bug and a 
sub-optimal "fix" was put in place to work around {{asVertex}}:

https://github.com/apache/incubator-tinkerpop/commit/9f804373e0450b961b4403161799bb3bc6b2dd06

That runtime check should be replaced when this issue is resolved.

> asVertex Does Not Always Return Correct State
> ---------------------------------------------
>
>                 Key: TINKERPOP3-783
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP3-783
>             Project: TinkerPop 3
>          Issue Type: Bug
>          Components: process
>    Affects Versions: 3.0.0-incubating
>            Reporter: stephen mallette
>            Assignee: Marko A. Rodriguez
>             Fix For: 3.0.1-incubating
>
>
> In {{AddPropertyStep}} the {{asVertex}} member variable is currently set as 
> follows:
> {code}
> this.asVertex = null != cardinality || this.vertexPropertyKeyValues.length > 
> 0;
> {code}
> which basically means that {{asVertex}} will get the "wrong" value if 
> {{property}} is called like this:
> {code}
> g.V().addV().property("this", "thing")
> {code}
> I would expect {{asVertex}} to return true in this case.  This value is 
> relied upon internal to the {{Step}} and by {{TraversalStrategy}} 
> implementations to determine the nature of what the the {{Step}} is 
> processing so it's important that it return properly under this condition.



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

Reply via email to