[
https://issues.apache.org/jira/browse/TINKERPOP-1541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15654066#comment-15654066
]
ASF GitHub Bot commented on TINKERPOP-1541:
-------------------------------------------
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/482
The only difference is `select(Pop.mix,...)` (that is `selectV3d2()`) and
there are 6 or so tests that test it in `RangeTest` and `TailTest`. I can
explicitly make them use the method `selectV3d2()`, but as you can see, its
simply just a call to `select(Pop.mix,...)`. I could make a `SelectV3d2Test`
and just copy those range tests over and call those methods explicitly, but
seems pointless as unlike `groupV320()`, its not a new step with different
semantics, its the same step with just a different argument.
> Select should default to Pop.last semantics
> -------------------------------------------
>
> Key: TINKERPOP-1541
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1541
> Project: TinkerPop
> Issue Type: Improvement
> Components: process
> Affects Versions: 3.2.3
> Reporter: Marko A. Rodriguez
> Assignee: Marko A. Rodriguez
> Labels: breaking
>
> Check this out:
> {code}
> gremlin> g.V().as('a').select('a').as('a').select('a')
> ==>[v[1],v[1]]
> ==>[v[2],v[2]]
> ==>[v[3],v[3]]
> ==>[v[4],v[4]]
> ==>[v[5],v[5]]
> ==>[v[6],v[6]]
> {code}
> Shouldn't we just return the uniques? This is a big decision as this can
> cause massive rippling breakage for users :).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
