Skip to content

chore: Bump to arrow 57#1402

Merged
kylebarron merged 2 commits into
geoarrow:mainfrom
ddupg:chore/arrow-57
Dec 4, 2025
Merged

chore: Bump to arrow 57#1402
kylebarron merged 2 commits into
geoarrow:mainfrom
ddupg:chore/arrow-57

Conversation

@ddupg
Copy link
Copy Markdown
Contributor

@ddupg ddupg commented Nov 27, 2025

closes #1401

@github-actions github-actions Bot added the chore label Nov 27, 2025
@ddupg ddupg mentioned this pull request Nov 27, 2025
3 tasks
@ddupg ddupg force-pushed the chore/arrow-57 branch 3 times, most recently from 04346f2 to 72f1c89 Compare November 28, 2025 07:01
Comment thread .cargo/config.toml Outdated
Comment thread js/Cargo.toml Outdated
arrow-wasm = { git = "https://github.com/kylebarron/arrow-wasm", rev = "6da94ef0a1522a244984a7d3d58a0339d0851d96" }
arrow-array = "57"
arrow-buffer = "57"
arrow-wasm = { git = "https://github.com/ddupg/arrow-wasm/", branch = "chore/arrow-57" }
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Comment thread python/geoarrow-core/src/interop/shapely/from_shapely.rs Outdated
Comment thread python/Cargo.toml Outdated
pyo3 = { version = "0.26", features = ["hashbrown", "serde", "anyhow"] }
pyo3-arrow = "0.12"
parquet = "57"
pyo3 = { version = "0.27", features = ["hashbrown", "serde", "anyhow"] }
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we split the pyo3 bump and the arrow bump into two separate PRs?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

According to the version compatibility matrix, arrow bump seems to require pyo3 and pyo3-arrow bump simultaneously. I tried bump arrow and pyo3 separately, but both resulted in dependency conflicts causing compilation errors. Please correct me if I'm wrong.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

According to the version compatibility matrix

Use v0.14 which uses latest arrow-rs but pyo3 0.26

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks. I rolled back the pyo3 bump, retaining only the arrow bump.

Comment thread rust/pyo3-geoarrow/src/scalar/mod.rs Outdated
Comment thread python/geoarrow-io/Cargo.toml Outdated
ddupg added 2 commits December 4, 2025 10:12
Change-Id: I4298d9078aa7c107ff9e787279ce27475e2e951b
Change-Id: Id17083c1870ac8d885cc824da500c0c7031ac71d
@ddupg ddupg marked this pull request as ready for review December 4, 2025 02:28
Copy link
Copy Markdown
Member

@kylebarron kylebarron left a comment

Choose a reason for hiding this comment

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

Thanks!

@kylebarron kylebarron merged commit f1deee4 into geoarrow:main Dec 4, 2025
10 checks passed
@kylebarron
Copy link
Copy Markdown
Member

If you're also interested in the pyo3 bump, we can do that as a separate PR

@ddupg
Copy link
Copy Markdown
Contributor Author

ddupg commented Dec 5, 2025

If you're also interested in the pyo3 bump, we can do that as a separate PR

Sure thing, I raised #1408 to bump pyo3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bump arrow to 57

3 participants