Skip to content

Comments

feat(java): add ReadOptions for read methods#6157

Merged
tisonkun merged 4 commits intoapache:mainfrom
liamzwbao:issue-5825-java-read-options
May 10, 2025
Merged

feat(java): add ReadOptions for read methods#6157
tisonkun merged 4 commits intoapache:mainfrom
liamzwbao:issue-5825-java-read-options

Conversation

@liamzwbao
Copy link
Contributor

Which issue does this PR close?

Closes #5825.

Rationale for this change

Mentioned in the above issue

What changes are included in this PR?

Support read options for both async and blocking read in java binding

Are there any user-facing changes?

Yes

@liamzwbao liamzwbao force-pushed the issue-5825-java-read-options branch from 59398ba to a06da4a Compare May 9, 2025 02:02
@liamzwbao liamzwbao force-pushed the issue-5825-java-read-options branch from a06da4a to 2224a76 Compare May 9, 2025 02:07
@liamzwbao liamzwbao marked this pull request as ready for review May 9, 2025 02:08
@liamzwbao liamzwbao requested review from Xuanwo and tisonkun as code owners May 9, 2025 02:08
@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 9, 2025
Copy link
Member

@tisonkun tisonkun left a comment

Choose a reason for hiding this comment

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

Great! Thanks for your contribution.

I'll try to make some code refactor on handling option fields.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label May 10, 2025
tisonkun added 2 commits May 10, 2025 12:39
Signed-off-by: tison <[email protected]>
Signed-off-by: tison <[email protected]>
@tisonkun tisonkun merged commit ffc82df into apache:main May 10, 2025
276 of 277 checks passed
@liamzwbao liamzwbao deleted the issue-5825-java-read-options branch May 14, 2025 03:01
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.

new feature: Add read with range support in java binding

2 participants