[
https://issues.apache.org/jira/browse/TINKERPOP-3083?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17876117#comment-17876117
]
ASF GitHub Bot commented on TINKERPOP-3083:
-------------------------------------------
kenhuuu commented on code in PR #2741:
URL: https://github.com/apache/tinkerpop/pull/2741#discussion_r1727975407
##########
docs/src/dev/provider/gremlin-semantics.asciidoc:
##########
@@ -1517,7 +1517,7 @@
link:https://tinkerpop.apache.org/docs/x.y.z/reference/#rTrim-step[reference]
*Arguments:*
* `separator` - The string character(s) used as delimiter to split the input
string. Nullable, a null separator will split on
-whitespaces.
+whitespaces. An empty separator will split on each character.
Review Comment:
Minor nit: I think it should probably be "empty string separator"
> The split() step should provide a way to split an entire string
> ---------------------------------------------------------------
>
> Key: TINKERPOP-3083
> URL: https://issues.apache.org/jira/browse/TINKERPOP-3083
> Project: TinkerPop
> Issue Type: Improvement
> Components: language, process
> Affects Versions: 3.7.2
> Reporter: Kelvin Lawrence
> Priority: Critical
>
> There are many use cases where it is helpful to be able to split an entire
> string into an array of characters. In other query languages, passing in a
> null string ('') as the parameter achieves this. I believe the Gremlin step
> should behave the same way. For example
> Current behavior
> {code:java}
> g.inject('Hello').split('')
> ['Hello']{code}
> Proposed new behavior
> {code:java}
> g.inject('Hello').split('')
> ['H','e','l','l','o']{code}
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
