Skip to content

feat(lister.go): Add opendal_list_options_set_limit and opendal_list_options_set_start_after#7649

Merged
dentiny merged 2 commits into
apache:mainfrom
FrankYang0529:opendal-7633
Jun 1, 2026
Merged

feat(lister.go): Add opendal_list_options_set_limit and opendal_list_options_set_start_after#7649
dentiny merged 2 commits into
apache:mainfrom
FrankYang0529:opendal-7633

Conversation

@FrankYang0529
Copy link
Copy Markdown
Member

Which issue does this PR close?

Closes #7633

Rationale for this change

The Go binding's List operation lacked support for limit and start_after options that are already available in the core Rust library. These options allow callers to paginate results and resume listing from a known position.

What changes are included in this PR?

C binding (bindings/c)

  • Added limit (uintptr_t) and start_after (char *) fields to opendal_list_options
  • Implemented opendal_list_options_set_limit and opendal_list_options_set_start_after C functions

Go binding (bindings/go)

  • Added ListWithLimit(limit uint) and ListWithStartAfter(startAfter string) option functions
  • Extended listOptions struct with the two new fields
  • Wired the new FFI calls (ffiListOptionsSetLimit, ffiListOptionsSetStartAfter) into Operator.List

Are there any user-facing changes?

Two new WithListFn option functions are added to the Go binding's public API:

op.List("prefix/", opendal.ListWithLimit(100))
op.List("prefix/", opendal.ListWithStartAfter("some/key"))

AI Usage Statement

OpenCode with claude-sonnet-4.6

…options_set_start_after

Signed-off-by: PoAn Yang <payang@apache.org>
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. releases-note/feat The PR implements a new feature or has a title that begins with "feat" labels May 31, 2026
Comment thread bindings/c/src/types.rs Outdated
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label May 31, 2026
@dentiny dentiny requested a review from yuchanns May 31, 2026 11:56
…set_start_after

Signed-off-by: PoAn Yang <payang@apache.org>
Copy link
Copy Markdown
Member

@dentiny dentiny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@dentiny dentiny merged commit 773eadb into apache:main Jun 1, 2026
102 checks passed
@FrankYang0529 FrankYang0529 deleted the opendal-7633 branch June 1, 2026 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/feat The PR implements a new feature or has a title that begins with "feat" size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(binding/go): Add ListWithLimit and ListWithStartAfter support

3 participants