[
https://issues.apache.org/jira/browse/TINKERPOP-2974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17946205#comment-17946205
]
ASF GitHub Bot commented on TINKERPOP-2974:
-------------------------------------------
kenhuuu opened a new pull request, #3098:
URL: https://github.com/apache/tinkerpop/pull/3098
https://issues.apache.org/jira/browse/TINKERPOP-2974
Not all implementations of Gremlin are going to be able to iterate the map
produced by PropertyMap step in order. This means that multiple by()s can't be
applied in order. Therefore, for those implementations, an exception should be
thrown instead.
<!--
Thanks for contributing! Reminders:
+ TARGET the earliest branch where you want the change
3.7-dev -> 3.7.4 (non-breaking only)
3.8-dev -> 3.8.0
master -> 4.0.0
+ Committers will MERGE the PR forward to newer versions
+ ADD entry to the CHANGELOG.asciidoc for the targeted version
Do not reference a JIRA number there
+ ADD JIRA number to title and link in description
+ PRs requires 3 +1s from committers OR
1 +1 and 7 day wait to merge.
+ MORE details: https://s.apache.org/rtnal
-->
> `valueMap` step returns properties in random order
> --------------------------------------------------
>
> Key: TINKERPOP-2974
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2974
> Project: TinkerPop
> Issue Type: Improvement
> Components: process
> Affects Versions: 3.5.6, 3.6.4
> Reporter: Valentyn Kahamlyk
> Priority: Minor
>
> `valueMap` step return properties in random order. In most cases it's not
> important, but
> but for example this does not allow using `by` modulators correctly.
> In this example I expect to get `name` unfolded, but modulator applied to
> `location`
> {code:java}
> gremlin> g = TinkerFactory.createTheCrew().traversal()
> ==>graphtraversalsource[tinkergraph[vertices:6 edges:14], standard]
> gremlin> g.V(1).valueMap("location","name").by().by(__.unfold())
> ==>[name:[marko],location:san diego]{code}
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
