[
https://issues.apache.org/jira/browse/TINKERPOP-3244?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18077795#comment-18077795
]
ASF GitHub Bot commented on TINKERPOP-3244:
-------------------------------------------
spmallette commented on PR #3416:
URL: https://github.com/apache/tinkerpop/pull/3416#issuecomment-4363617916
Thanks for the PR. I'll leave it to the Go experts to dig into the details
here but I'm interested in the signoff since I recently added that instruction
to our AGENTS setup:
> Assisted-by: Claude:claude-opus-4-7 [Claude Code]
It signed like this too for me yesterday as a first time using that
instruction. I'd envisioned:
> Assisted-by: Claude Code:claude-opus-4-7
I might need to better define that to get what I want there. It's
interpreting the "tool" to be Claude Code - maybe because the tool is not seen
as optional??
> 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)