Skip to content

feat(integrations/parquet): upgrade arrow-rs to 58.0.0#7250

Merged
asukaminato0721 merged 2 commits intoapache:mainfrom
jo-migo:arrow-rs-58-0-0
Mar 20, 2026
Merged

feat(integrations/parquet): upgrade arrow-rs to 58.0.0#7250
asukaminato0721 merged 2 commits intoapache:mainfrom
jo-migo:arrow-rs-58-0-0

Conversation

@jo-migo
Copy link
Copy Markdown
Contributor

@jo-migo jo-migo commented Mar 18, 2026

Which issue does this PR close?

Closes #7249

Rationale for this change

I would like to use both parquet_opendal and the latest version of arrow-rs in a project I maintain. Currently this is blocked by the pinned dependency of parquet_opendal on parquet = "54.2".

What changes are included in this PR?

Update AsyncFileReader impl to the new API: get_metadata now accepts ArrowReaderOptions, and ranges use u64 instead of usize throughout, removing the need for manual casts.

Are there any user-facing changes?

No

Update AsyncFileReader impl to the new API: get_metadata now accepts
ArrowReaderOptions, and ranges use u64 instead of usize throughout,
removing the need for manual casts.
@jo-migo jo-migo requested review from Xuanwo and tisonkun as code owners March 18, 2026 15:37
@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. releases-note/feat The PR implements a new feature or has a title that begins with "feat" labels Mar 18, 2026
Copy link
Copy Markdown
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.

LGTM.

Would you fix the code-format CI issue?

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Mar 18, 2026
@jo-migo
Copy link
Copy Markdown
Contributor Author

jo-migo commented Mar 19, 2026

Thanks for the review @tisonkun 😄 I fixed it.

@asukaminato0721 asukaminato0721 merged commit ef67dcf into apache:main Mar 20, 2026
34 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:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

parquet_opendal is incompatible with latest parquet/arrow-rs (58.x)

3 participants