Skip to content

chore(rust): bump edition to 2024, MSRV to 1.85, and update dependencies#4043

Draft
if0ne wants to merge 3 commits intoapache:mainfrom
if0ne:rust/bump-version
Draft

chore(rust): bump edition to 2024, MSRV to 1.85, and update dependencies#4043
if0ne wants to merge 3 commits intoapache:mainfrom
if0ne:rust/bump-version

Conversation

@if0ne
Copy link
Contributor

@if0ne if0ne commented Mar 4, 2026

Changes

  • Rust edition 2021 -> 2024 with corresponding source code fixes for new edition requirements
  • MSRV 1.81 -> 1.85 across the Rust workspace
  • Arrow crates 57.x -> 58.x (lower bound tightened from >=53.1.0 to >=58)

Signed-off-by: if0ne <pavel.agafonov.al@gmail.com>
@if0ne if0ne requested a review from wjones127 as a code owner March 4, 2026 08:24
@if0ne if0ne marked this pull request as draft March 4, 2026 08:24
rust/Cargo.toml Outdated
edition = "2021"
rust-version = "1.81"
edition = "2024"
rust-version = "1.88"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why choose 1.88?
The constraint by arrow seems to be 1.85.
apache/arrow-rs@9d0e8be

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Initially, I chose 1.88 because adbc_datafusion has an MSRV of 1.88, and the cargo check produced an error when the 1.85 toolchain was selected.

if0ne added 2 commits March 4, 2026 15:45
Signed-off-by: if0ne <pavel.agafonov.al@gmail.com>
Signed-off-by: if0ne <pavel.agafonov.al@gmail.com>
@if0ne if0ne force-pushed the rust/bump-version branch from af2f86f to 8ea0a46 Compare March 4, 2026 13:24
@eitsupi
Copy link
Contributor

eitsupi commented Mar 4, 2026

I suppose some deprecation warnings can be removed.
See #3894

@if0ne if0ne changed the title chore(rust): bump edition to 2024, MSRV to 1.88, and update dependencies chore(rust): bump edition to 2024, MSRV to 1.85, and update dependencies Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants