Skip to content

chore(rust/driver/datafusion): bump dependent arrow to 58.3.0#4316

Merged
lidavidm merged 2 commits into
apache:mainfrom
eitsupi:rust/bump-arrow
May 14, 2026
Merged

chore(rust/driver/datafusion): bump dependent arrow to 58.3.0#4316
lidavidm merged 2 commits into
apache:mainfrom
eitsupi:rust/bump-arrow

Conversation

@eitsupi
Copy link
Copy Markdown
Contributor

@eitsupi eitsupi commented May 13, 2026

Supersede #4313

@eitsupi eitsupi requested a review from wjones127 as a code owner May 13, 2026 14:12
Copy link
Copy Markdown
Member

@lidavidm lidavidm left a comment

Choose a reason for hiding this comment

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

Thanks!

Is a bound like >=58.1,<59 workable? Or is that discouraged?

@eitsupi
Copy link
Copy Markdown
Contributor Author

eitsupi commented May 13, 2026

Is a bound like >=58.1,<59 workable? Or is that discouraged?

I think simply 58.1 would be sufficient.
https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#default-requirements

Would it be clearer to simply update Cargo.toml along with the lock file?

@lidavidm
Copy link
Copy Markdown
Member

Ah, yeah, if we can just have a bound like "58" then that would be better, IMO

Copy link
Copy Markdown
Member

@lidavidm lidavidm left a comment

Choose a reason for hiding this comment

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

Thanks!

@lidavidm lidavidm merged commit 5546901 into apache:main May 14, 2026
19 checks passed
@eitsupi eitsupi deleted the rust/bump-arrow branch May 14, 2026 16:18
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