Skip to content

fix: align arrow version with spark and add docker integration tests#205

Merged
jackye1995 merged 6 commits intolance-format:mainfrom
hamersaw:feature/docker-tests
Feb 6, 2026
Merged

fix: align arrow version with spark and add docker integration tests#205
jackye1995 merged 6 commits intolance-format:mainfrom
hamersaw:feature/docker-tests

Conversation

@hamersaw
Copy link
Copy Markdown
Collaborator

@hamersaw hamersaw commented Feb 5, 2026

The recent upgrade of arrow version (15.0 -> 18.3) in lance-core does not play well with Spark 3.5. This results in failures within the Lance Spark connector for simple operations (ex. CREATE TABLE). In this PR we pin the arrow version to 15.0 (used for ~2 years) for all 3.5 and 3.4 profiles. The other alternative approaches are to (1) downgrade lance-core arrow dependency or (2) maintain separate lance-core branches. This seems like the most reasonable approach.

Future work will be to integrate these into our CI pipeline.

Closes: #196

Signed-off-by: Daniel Rammer <hamersaw@protonmail.com>
@github-actions github-actions Bot added the ci Features related to CI improvements label Feb 5, 2026
Comment thread docker/Dockerfile
Copy link
Copy Markdown
Contributor

@jackye1995 jackye1995 left a comment

Choose a reason for hiding this comment

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

We should make sure we run this in CI

Signed-off-by: Daniel Rammer <hamersaw@protonmail.com>
Signed-off-by: Daniel Rammer <hamersaw@protonmail.com>
Signed-off-by: Daniel Rammer <hamersaw@protonmail.com>
@hamersaw hamersaw marked this pull request as ready for review February 6, 2026 20:27
Comment thread lance-spark-3.4_2.12/pom.xml
Comment thread pom.xml Outdated
Comment thread Makefile Outdated
Signed-off-by: Daniel Rammer <hamersaw@protonmail.com>
Signed-off-by: Daniel Rammer <hamersaw@protonmail.com>
Copy link
Copy Markdown
Contributor

@jackye1995 jackye1995 left a comment

Choose a reason for hiding this comment

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

thanks for the work!

@jackye1995 jackye1995 changed the title ci: add docker integration tests fix: align arrow version with spark and add docker integration tests Feb 6, 2026
@github-actions github-actions Bot added the bug Something isn't working label Feb 6, 2026
@jackye1995 jackye1995 merged commit dc2ff90 into lance-format:main Feb 6, 2026
15 checks passed
@hamersaw hamersaw deleted the feature/docker-tests branch February 6, 2026 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci Features related to CI improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

EMR Spark 3.5 Arrow version conflict

2 participants