Skip to content

Comments

feat(bindings/cpp): cpp async op && reader, lister#6228

Merged
asukaminato0721 merged 18 commits intoapache:mainfrom
asukaminato0721:cpp-async-list
Jun 16, 2025
Merged

feat(bindings/cpp): cpp async op && reader, lister#6228
asukaminato0721 merged 18 commits intoapache:mainfrom
asukaminato0721:cpp-async-list

Conversation

@asukaminato0721
Copy link
Contributor

@asukaminato0721 asukaminato0721 commented May 28, 2025

Which issue does this PR close?

part of #5194 .
part of #2981

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

support following async api.

list
exists
create_dir
delete
remove_all

and reader, lister

@asukaminato0721 asukaminato0721 changed the title Cpp async op feat(bindings/cpp): cpp async op May 28, 2025
@asukaminato0721 asukaminato0721 marked this pull request as ready for review May 28, 2025 15:35
@asukaminato0721 asukaminato0721 requested a review from Xuanwo as a code owner May 28, 2025 15:35
@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" size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels May 28, 2025
@asukaminato0721 asukaminato0721 changed the title feat(bindings/cpp): cpp async op feat(bindings/cpp): cpp async op && reader, lister May 28, 2025
@asukaminato0721 asukaminato0721 force-pushed the cpp-async-list branch 2 times, most recently from 87f8d9f to 78d4842 Compare May 29, 2025 12:49
@asukaminato0721 asukaminato0721 force-pushed the cpp-async-list branch 2 times, most recently from f0ef521 to 55aa344 Compare June 3, 2025 09:59
@asukaminato0721 asukaminato0721 force-pushed the cpp-async-list branch 3 times, most recently from 5e76347 to 734a314 Compare June 10, 2025 11:06
Copy link
Member

@silver-ymz silver-ymz left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 16, 2025
@asukaminato0721 asukaminato0721 merged commit 5bfd360 into apache:main Jun 16, 2025
34 checks passed
@asukaminato0721 asukaminato0721 deleted the cpp-async-list branch June 16, 2025 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bindings/cpp 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:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants