[ https://issues.apache.org/jira/browse/TINKERPOP-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16027922#comment-16027922 ]
pieter martin commented on TINKERPOP-1681: ------------------------------------------ Just to check does your fix also work for, {noformat} 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(a.id()).hasId(b.id()).toList(); Assert.assertTrue(vertices.isEmpty()); {noformat} Regarding {{g.V().not(identity())}} it for sure is bad as {{not(identity())}} is not the kind of thing that Sqlg at least, even tries to optimize. > 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)