Skip to content

build(deps): bump vortex-jni from 0.78.0 to 0.79.0#286

Merged
dfa1 merged 1 commit into
mainfrom
deps/vortex-jni-0.79.0
Jul 19, 2026
Merged

build(deps): bump vortex-jni from 0.78.0 to 0.79.0#286
dfa1 merged 1 commit into
mainfrom
deps/vortex-jni-0.79.0

Conversation

@dfa1

@dfa1 dfa1 commented Jul 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • Bumps dev.vortex:vortex-jni 0.78.0 → 0.79.0 (supersedes dependabot's build(deps): Bump dev.vortex:vortex-jni from 0.78.0 to 0.79.0 #282, which was red).
  • 0.79.0 changed the Arrow vector type the JNI reference exports for Utf8 columns from VarCharVector to ViewVarCharVector. JavaWritesRustReadsIntegrationTest hard-cast to VarCharVector at three Arrow-read call sites, breaking every string-column interop test with a ClassCastException.
  • Fix: widen those call sites to the common ValueVector interface (getObject/isNull are both declared there), no per-vector-type branching needed.

Test plan

  • Targeted rerun of the two previously-failing tests + readStringColumn helper: pass
  • Full JavaWritesRustReadsIntegrationTest class: 217/217 pass, 2 skipped (unrelated)
  • Full integration module verify: 316/316 pass, 0 failures, 4 skipped (unrelated)

🤖 Generated with Claude Code

0.79.0 changed the Arrow vector type the JNI reference exports for
Utf8 columns from VarCharVector to ViewVarCharVector (Arrow's Utf8View
layout). JavaWritesRustReadsIntegrationTest hard-cast to VarCharVector
at its three Arrow-read call sites, breaking every string-column
interop test with a ClassCastException.

Widen those call sites to the common ValueVector interface — getObject
and isNull are both declared there, so no per-vector-type branching is
needed. Verified full integration module (316 tests, 0 failures) and
the specific string-column tests that were failing on ubuntu/macos CI.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@dfa1
dfa1 merged commit 85e7d4e into main Jul 19, 2026
6 checks passed
@dfa1
dfa1 deleted the deps/vortex-jni-0.79.0 branch July 19, 2026 08:26
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.

1 participant