spmallette opened a new pull request #1266: TINKERPOP-2351 Fixed bug in Order when enum is a key in Map URL: https://github.com/apache/tinkerpop/pull/1266 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
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
