[
https://issues.apache.org/jira/browse/TINKERPOP-2996?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17781432#comment-17781432
]
ASF GitHub Bot commented on TINKERPOP-2996:
-------------------------------------------
vkagamlyk opened a new pull request, #2316:
URL: https://github.com/apache/tinkerpop/pull/2316
Current implementation of Golang Translator can't handle array arguments.
For example `g.inject([[1,2],[3,4]])` translated as `g.Inject(1,2,3,4)' what
is wrong.
Also in this PR `P.Within` and `P.Without` extended to accept array
arguments similar to other GLV's.
https://issues.apache.org/jira/browse/TINKERPOP-2996
> Golang Translator in core does not properly translate list arguments
> --------------------------------------------------------------------
>
> Key: TINKERPOP-2996
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2996
> Project: TinkerPop
> Issue Type: Bug
> Components: go
> Affects Versions: 3.7.0, 3.5.7, 3.6.5
> Reporter: Yang Xia
> Priority: Critical
>
> Currently for the bytecode translatro in core, all list type arguments are
> translated as varargs for golang, instead of proper list type like
> []interface{}{}.
> Currently this is causing errors in certain translations, i.e. g.inject([1,
> 2]) is being translated to g.Inject(1, 2), but some investigation will be
> needed to tease apart cases that may need a varargs argument.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
