[ https://issues.apache.org/jira/browse/TINKERPOP-2978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17778851#comment-17778851 ]
ASF GitHub Bot commented on TINKERPOP-2978: ------------------------------------------- kenhuuu commented on code in PR #2302: URL: https://github.com/apache/tinkerpop/pull/2302#discussion_r1369372021 ########## docs/src/reference/the-traversal.asciidoc: ########## @@ -3759,12 +3885,14 @@ link:++https://tinkerpop.apache.org/javadocs/x.y.z/core/org/apache/tinkerpop/gre [[reverse-step]] === Reverse Step -The `reverse()`-step (*map*) returns a reversed string of the incoming string traverser. Null values are not processed and remain -as null when returned. If the incoming traverser is a non-String value then an `IllegalArgumentException` will be thrown. +The `reverse()`-step (*map*) returns a reversed of the incoming list traverser. Single values (including null) are not +processed and are added back to the Traversal Stream unchanged. If the incoming traverser is a String value then the +reversed String will be returned. [gremlin-groovy,modern] ---- g.V().values("name").reverse() +g.V().values("name").fold().reverse() Review Comment: Done. ########## docs/src/upgrade/release-3.7.x.asciidoc: ########## @@ -225,11 +226,34 @@ gremlin> g.V().values("age").fold().all(P.gt(10)) ==>[29,27,32,35] gremlin> g.V().values("age").fold().any(P.eq(32)) ==>[29,27,32,35] +gremlin> g.V().values("age").fold().product(__.V().values("age").limit(2).fold()) +==>[[29,29],[29,27],[27,29],[27,27],[32,29],[32,27],[35,29],[35,27]] +gremlin> g.V().values("age").fold().merge([32,30,50]) +==>[32,50,35,27,29,30] +gremlin> g.V().values("age").fold().combine([32,30,50]) +==>[29,27,32,35,32,30,50] +gremlin> g.V().values("age").fold().intersect([32,30,50]) +==>[32] +gremlin> g.V().values("age").fold().disjunct([32,30,50]) +==>[50,35,27,29,30] +gremlin> g.V().values("age").fold().difference([32,30,50]) +==>[35,27,29] +gremlin> g.V().values("age").order().by(desc).fold().reverse() +==>[27,29,32,35] +gremlin> g.V().values("age").fold().join("-") Review Comment: conjoin. updated. > Add List Manipulation Steps to Gremlin > -------------------------------------- > > Key: TINKERPOP-2978 > URL: https://issues.apache.org/jira/browse/TINKERPOP-2978 > Project: TinkerPop > Issue Type: Improvement > Components: language > Reporter: Cole Greer > Priority: Major > > Today Gremlin requires that users fall back to closures to handle many common > list manipulation options that users want to do on data in the graph. This > is a problem for many users as many of the providers prevent the use of > closures due to the security risks so for these users there is no way to > manipulate lists directly. > A full list of proposed functions and semantics is detailed here: > https://github.com/apache/tinkerpop/blob/3.7.0/docs/src/dev/future/proposal-3-remove-closures.asciidoc -- This message was sent by Atlassian Jira (v8.20.10#820010)