[ 
https://issues.apache.org/jira/browse/TINKERPOP-3248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18118654#comment-18118654
 ] 

ASF GitHub Bot commented on TINKERPOP-3248:
-------------------------------------------

kenhuuu commented on code in PR #3673:
URL: https://github.com/apache/tinkerpop/pull/3673#discussion_r4089957020


##########
docs/src/reference/gremlin-variants.asciidoc:
##########
@@ -729,8 +729,6 @@ therefore cardinality functions that take a value like 
`list()`, `set()`, and `s
 [[gremlin-go-limitations]]
 === Limitations
 
-* The Gremlin `Character` type is not supported by Gremlin-Go.

Review Comment:
   You might want to write here how the new Char type behaves. That is, users 
of gremlin-go that try something like `g.inject('a')` will lead to it being 
intepreted as an Integer. They have to use the new Char type that wraps a Rune 
to get the GremlinLang to actually generate a Char type.





> Character support in gremlin-go
> -------------------------------
>
>                 Key: TINKERPOP-3248
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-3248
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: go
>    Affects Versions: 3.8.1
>            Reporter: Ken Hu
>            Priority: Major
>
> gremlin-go doesn't support the Character right now but it should be 
> representable with Rune. Add this to both GremlinLang and GraphBinary in 
> gremlin-go.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to