Skip to content

ci: support arrow-js v21#655

Merged
bednar merged 2 commits intomainfrom
ci/support-arrowjs-v21
Jan 9, 2026
Merged

ci: support arrow-js v21#655
bednar merged 2 commits intomainfrom
ci/support-arrowjs-v21

Conversation

@NguyenHoangSon96
Copy link
Contributor

@NguyenHoangSon96 NguyenHoangSon96 commented Jan 9, 2026

Closes Dependabot PR

  • Arrow-js v21 has added support for reading compressed arrow IPC stream PR. They will check if the RecordBatch.compression property is null or not
Screenshot 2026-01-09 at 17 40 53

In our current integration test, we send an empty message back to the client, but we did not set the "compression" property of the message to "null" explicitly, so it causes an error.

Proposed Changes

_ Explicitly set the property "compression" of the return message to "null".

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • A test has been added if appropriate
  • Tests pass
  • Commit messages are conventional
  • Sign CLA (if not already signed)

@NguyenHoangSon96 NguyenHoangSon96 self-assigned this Jan 9, 2026
@NguyenHoangSon96 NguyenHoangSon96 changed the title ci: support arrowjs v21 ci: support arrow-js v21 Jan 9, 2026
@bednar bednar merged commit ee2a8e9 into main Jan 9, 2026
16 checks passed
@bednar bednar deleted the ci/support-arrowjs-v21 branch January 9, 2026 11:00
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