[ https://issues.apache.org/jira/browse/TINKERPOP-1219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15199444#comment-15199444 ]
ASF GitHub Bot commented on TINKERPOP-1219: ------------------------------------------- Github user okram commented on the pull request: https://github.com/apache/incubator-tinkerpop/pull/267#issuecomment-197865254 @spmallette -- the problem is that this is a breaking change in that `HasContainers` do id magic. Why do they do that? Seems like a really really bad idea to have something so "high level" do id manipulations. If you are aware of this and have thought it all through and still thing this PR is good, okay. > Create a test case that ensures the provider's compilation of g.V(x) and > g.V().hasId(x) is identical > ---------------------------------------------------------------------------------------------------- > > Key: TINKERPOP-1219 > URL: https://issues.apache.org/jira/browse/TINKERPOP-1219 > Project: TinkerPop > Issue Type: Improvement > Components: test-suite > Affects Versions: 3.1.1-incubating > Reporter: Mike Dias > Assignee: Marko A. Rodriguez > Priority: Minor > Labels: breaking, test > Fix For: 3.1.2-incubating > > > As discussed in this topic: > https://groups.google.com/forum/#!topic/gremlin-users/Kz2bOeAeqh4 > It will ensure the same behavior between g.V().hasId(id) vs. g.V(id) and > graph.vertices(id) -- This message was sent by Atlassian JIRA (v6.3.4#6332)