https://issues.apache.org/jira/browse/TINKERPOP-2351

This seemed like an easy fix. Can't think of any bad side-effects. Not sure if 
there are other special cases we should consider, but at least this common case 
is solved.

```text
gremlin> g.V().valueMap(true).order(local).by(keys,desc)
==>[name:[marko],label:person,id:1,age:[29]]
==>[name:[vadas],label:person,id:2,age:[27]]
==>[name:[lop],lang:[java],label:software,id:3]
==>[name:[josh],label:person,id:4,age:[32]]
==>[name:[ripple],lang:[java],label:software,id:5]
==>[name:[peter],label:person,id:6,age:[35]]
gremlin> g.V(1).valueMap(true).order(local).by(keys,desc).next()[T.id]
==>1
gremlin> g.V(1).valueMap(true).order(local).by(keys,desc).next()[T.label]
==>person
```

When this merges forward to `3.4-dev` I will modify the test to use 
`elementMap()` rather than `valueMap(true)` since that method is deprecated.

All tests pass with `docker/build.sh -t -i`

VOTE +1

[ Full content available at: https://github.com/apache/tinkerpop/pull/1266 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to