Skip to content

Upgrade Arrow from 22.0.0 to 23.0.0#169

Merged
timosachsenberg merged 2 commits into
OpenMS:masterfrom
timosachsenberg:arrow-23
Feb 6, 2026
Merged

Upgrade Arrow from 22.0.0 to 23.0.0#169
timosachsenberg merged 2 commits into
OpenMS:masterfrom
timosachsenberg:arrow-23

Conversation

@timosachsenberg
Copy link
Copy Markdown
Contributor

@timosachsenberg timosachsenberg commented Feb 6, 2026

Summary

  • Upgrade Apache Arrow from 22.0.0 to 23.0.0
  • Source archive uploaded to contrib-sources release 3.6

OpenMS now requires Arrow 23+ for parquet file format compatibility (repetition level histogram changes). This updates the contrib to build Arrow 23.

After merge

Rebuild manylinux container images and Windows contrib tarball.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated Arrow library to version 23.0.0.
    • Updated the base download URL for contrib archives to the 3.6.0 location.

OpenMS now requires Arrow 23+ for parquet file format compatibility
(repetition level histogram changes in Arrow 23). The source archive
has been uploaded to contrib-sources.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 6, 2026

📝 Walkthrough

Walkthrough

Updated Apache Arrow dependency references in CMakeLists.txt (22.0.0 → 23.0.0) and changed the contrib download base URL in macros.cmake (3.5.0_pre_2 → 3.6.0), including updated archive filenames and checksum.

Changes

Cohort / File(s) Summary
Arrow library declarations
CMakeLists.txt
Bumped Arrow version to 23.0.0: updated ARROW_DIR, ARCHIVE_ARROW, ARCHIVE_ARROW_TAR, and ARCHIVE_ARROW_SHA256.
Contrib download base URL
cmake/macros.cmake
Updated BASE_URL used by download_contrib_archive from 3.5.0_pre_2 to 3.6.0; download flow and validations unchanged.

Possibly related PRs

Poem

🐰
A rabbit hops through CMake trees,
Tucking Arrow's new release with ease.
URLs adjusted, checksums true,
Builds await the version new.
Hooray — a tiny upgrade breeze!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: upgrading Apache Arrow from version 22.0.0 to 23.0.0, which is directly reflected in the CMakeLists.txt updates and the PR objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@timosachsenberg timosachsenberg merged commit 60b3d74 into OpenMS:master Feb 6, 2026
1 check passed
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