Skip to content

Comments

feat(bindings/c): Add seek support for c-binding #6119

Merged
Xuanwo merged 4 commits intoapache:mainfrom
mzygQAQ:c-binding-seek
Apr 29, 2025
Merged

feat(bindings/c): Add seek support for c-binding #6119
Xuanwo merged 4 commits intoapache:mainfrom
mzygQAQ:c-binding-seek

Conversation

@mzygQAQ
Copy link
Contributor

@mzygQAQ mzygQAQ commented Apr 28, 2025

Which issue does this PR close?

Closes #6083.

Rationale for this change

What changes are included in this PR?

Add seek support in c-binding.

Are there any user-facing changes?

Your Name added 2 commits April 29, 2025 00:17
@mzygQAQ mzygQAQ requested a review from xyjixyjixyji as a code owner April 28, 2025 16:25
@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 Apr 28, 2025
@mzygQAQ mzygQAQ changed the title Add seek support for c-binding feat(bindings/c): Add seek support for c-binding Apr 28, 2025
@Xuanwo Xuanwo requested a review from silver-ymz April 29, 2025 01:57
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 Apr 29, 2025
@Xuanwo Xuanwo merged commit f13ae40 into apache:main Apr 29, 2025
69 checks passed
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.

Need range reader or seek in c binding

3 participants