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

ASF GitHub Bot commented on TINKERPOP-1681:
-------------------------------------------

Github user okram commented on the issue:

    https://github.com/apache/tinkerpop/pull/617
  
    Seems right to me. Just to be certain `x.y.z` are ANDs and before, for 
`hasLabel()` and `hasId()` they were being OR'd. Test cases make sense.
    
    VOTE +1.


> Multiple hasId's are or'd into GraphStep
> ----------------------------------------
>
>                 Key: TINKERPOP-1681
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1681
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: process
>    Affects Versions: 3.2.4
>            Reporter: Daniel Kuppitz
>            Assignee: Daniel Kuppitz
>
> From the dev mailing list:
> {noformat}
>     @Test
>     public void testHasId() {
>         final TinkerGraph graph = TinkerGraph.open();
>         Vertex a = graph.addVertex(T.label, "A");
>         Vertex b = graph.addVertex(T.label, "B");
>         List<Vertex> vertices = 
> graph.traversal().V().hasId(a.id()).hasId(b.id()).toList();
>         Assert.assertTrue(vertices.isEmpty());
>     }
> {noformat}
> The test fails as the both vertices are returned.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to