[ 
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)

Reply via email to