[
https://issues.apache.org/jira/browse/TINKERPOP-3244?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18077877#comment-18077877
]
ASF GitHub Bot commented on TINKERPOP-3244:
-------------------------------------------
L0Lmaker commented on PR #3416:
URL: https://github.com/apache/tinkerpop/pull/3416#issuecomment-4365589637
Thanks for the review @Cole-Greer. Pushed `584ada5d4d` to address both
points:
- Go translators in `gremlin-core` and `gremlin-javascript` now emit
`NextN(n)` for the `next(n)` terminal step (no-arg `next()` continues to map to
`Next()`). Implemented as a `visitTraversalTerminalMethod_next` override in
each translator, since the symbol-mapping layer doesn't see the
integer-argument distinction. Existing parameterized test rows updated.
- Added a short paragraph in `gremlin-variants.asciidoc` under the Go `===
Differences` section explaining why the batched form is exposed as a sibling
method.
- Expanded the CHANGELOG bullet to also name the translator updates,
matching the prior `gremlin-go` + `GoTranslatorVisitor` entry already in the
4.0.0 section.
> Add next(n) batch iteration to gremlin-go for GLV parity
> --------------------------------------------------------
>
> Key: TINKERPOP-3244
> URL: https://issues.apache.org/jira/browse/TINKERPOP-3244
> Project: TinkerPop
> Issue Type: Improvement
> Components: driver
> Affects Versions: 3.7.5, 3.8.1
> Reporter: Taylor Riggan
> Priority: Minor
>
> The Java, Python, and .NET GLVs all support `next\(n\)` on Traversal to
> retrieve multiple results at once, but gremlin-go only supports single-item
> `Next()` returning `(*Result, error)`. This creates an API inconsistency
> across GLVs that makes it harder to write portable examples and documentation.
> Currently, users must manually collect results from `GetResultSet()` +
> `One()` in a loop to achieve the same behavior. A `Next(n int) ([]*Result,
> error)` method on `Traversal` would bring gremlin-go in line with the other
> GLVs.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
