[ https://issues.apache.org/jira/browse/TINKERPOP-2951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17747232#comment-17747232 ]
ASF GitHub Bot commented on TINKERPOP-2951: ------------------------------------------- Cole-Greer commented on PR #2157: URL: https://github.com/apache/tinkerpop/pull/2157#issuecomment-1650861179 Hi Itay, I wanted to follow up on my previous comment and build off of what Valentyn's comment. As Valentyn mentioned this will require support for additional types and more testing. I also think it would be nice if the tests were to use a `GraphTraversal` to construct the `Bytecode` similar to what is done in the python translator tests, I believe this would make the tests easier to read and write. Let me know if you need any help with this. There is a bit of a time crunch if you want to get this in our upcoming release as we are currently in code freeze. I think all of the python translator tests will need to be translated over and need to pass before we can proceed here. Also it would ideally be completed tomorrow to avoid jeopardizing the release. Please let me know if you have any questions regarding this. I will make a best effort to help get this included in the release but due to the timing it cannot be guaranteed. > 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)