Skip to content

Comments

fix(bindings/nodejs): use value.prefetch instead of value.concurrent for prefetch option#6502

Merged
Xuanwo merged 1 commit intoapache:mainfrom
kingsword09:fix-nodejs-readeroptions
Aug 12, 2025
Merged

fix(bindings/nodejs): use value.prefetch instead of value.concurrent for prefetch option#6502
Xuanwo merged 1 commit intoapache:mainfrom
kingsword09:fix-nodejs-readeroptions

Conversation

@kingsword09
Copy link
Contributor

Which issue does this PR close?

Related #6449.

Rationale for this change

Fixed the prefetch option in ReaderOptions by using value.prefetch instead of value.concurrent.

What changes are included in this PR?

Are there any user-facing changes?

@kingsword09 kingsword09 requested a review from suyanhanx as a code owner August 12, 2025 02:29
@kingsword09 kingsword09 requested review from Xuanwo and removed request for suyanhanx August 12, 2025 02:29
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. releases-note/fix The PR fixes a bug or has a title that begins with "fix" labels Aug 12, 2025
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thank you for the fix!

@Xuanwo Xuanwo merged commit 542df66 into apache:main Aug 12, 2025
65 checks passed
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 12, 2025
@kingsword09 kingsword09 deleted the fix-nodejs-readeroptions branch August 12, 2025 02: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/fix The PR fixes a bug or has a title that begins with "fix" size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants