[
https://issues.apache.org/jira/browse/TINKERPOP-2951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17747179#comment-17747179
]
ASF GitHub Bot commented on TINKERPOP-2951:
-------------------------------------------
vkagamlyk commented on PR #2157:
URL: https://github.com/apache/tinkerpop/pull/2157#issuecomment-1650581833
Hi Itay,
Thanks for your contribution!
This PR is missing support for some types like integers other then `int`,
bool, dates, collections, predicates, strategies, etc.
For comprehensive list of test cases, please take a look [at translator
tests for
Python](https://github.com/apache/tinkerpop/blob/master/gremlin-python/src/main/python/tests/process/test_translator.py)
or other GLV implementation.
Would also be nice to mention this PR changes in
[changelog](https://github.com/apache/tinkerpop/blob/master/CHANGELOG.asciidoc)
> Add translator to the Go GLV
> ----------------------------
>
> Key: TINKERPOP-2951
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2951
> Project: TinkerPop
> Issue Type: Improvement
> Components: go
> Affects Versions: 3.5.6, 3.6.4
> Reporter: Valentyn Kahamlyk
> Priority: Major
>
> Currently there is no equivalent to GroovyTranslator for the Go client. There
> are Translator's in all other GLV's. Adding one for the Go client will
> further help in moving all the clients to a consistent level of
> functionality.
> https://tinkerpop.apache.org/docs/current/reference/#translators
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
