Skip to content

chore(C++): update the arrow version to latest 23.0.0#853

Merged
yangxk1 merged 1 commit intoapache:mainfrom
SYaoJun:0211_update_arrow_version
Feb 24, 2026
Merged

chore(C++): update the arrow version to latest 23.0.0#853
yangxk1 merged 1 commit intoapache:mainfrom
SYaoJun:0211_update_arrow_version

Conversation

@SYaoJun
Copy link
Copy Markdown
Contributor

@SYaoJun SYaoJun commented Feb 11, 2026

Reason for this PR

The latest version of Apache Arrow is 23.0.0. This new version offers improved efficiency and numerous optimized features that would benefit us. How about updating the dependency to the latest version?

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.21%. Comparing base (02bd3fe) to head (63513d0).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #853      +/-   ##
============================================
- Coverage     80.21%   80.21%   -0.01%     
  Complexity      615      615              
============================================
  Files            93       93              
  Lines         10257    10258       +1     
  Branches       1049     1050       +1     
============================================
  Hits           8228     8228              
- Misses         1789     1790       +1     
  Partials        240      240              
Flag Coverage Δ
cpp 71.50% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Sober7135 Sober7135 changed the title chore(cmake): update the cmake version to latest 23.0.0 chore(C++): update the arrow version to latest 23.0.0 Feb 12, 2026
@Sober7135
Copy link
Copy Markdown
Contributor

Nice work! Could you also add a CMake option like GRAPHAR_ARROW_VERSION so we can test against different Arrow versions by setting it?

@SYaoJun
Copy link
Copy Markdown
Contributor Author

SYaoJun commented Feb 12, 2026

Nice work! Could you also add a CMake option like GRAPHAR_ARROW_VERSION so we can test against different Arrow versions by setting it?

Ok, I will update as soon as possible.

@SYaoJun SYaoJun marked this pull request as draft February 12, 2026 14:10
@SYaoJun SYaoJun force-pushed the 0211_update_arrow_version branch from d6f7b6c to a969735 Compare February 13, 2026 15:22
@SYaoJun SYaoJun marked this pull request as ready for review February 13, 2026 15:43
@SYaoJun
Copy link
Copy Markdown
Contributor Author

SYaoJun commented Feb 13, 2026

I also added some flags on macOS to support Arrow 23.0.0. Additionally, I included independent components introduced in Arrow 23.0.0, such as arrow_compute

@yangxk1
Copy link
Copy Markdown
Contributor

yangxk1 commented Feb 14, 2026

Good job! Howere why do the cpp checks have been skipped

@SYaoJun
Copy link
Copy Markdown
Contributor Author

SYaoJun commented Feb 14, 2026

Good job! Howere why do the cpp checks have been skipped

CI may be triggered when cpp files changed. I can pull again later.

@yangxk1
Copy link
Copy Markdown
Contributor

yangxk1 commented Feb 14, 2026

It seems that GitHub action still thinks it is a draft

@SYaoJun SYaoJun force-pushed the 0211_update_arrow_version branch from a969735 to f3485d3 Compare February 14, 2026 11:18
@SYaoJun SYaoJun force-pushed the 0211_update_arrow_version branch from f3485d3 to 63513d0 Compare February 14, 2026 11:22
Copy link
Copy Markdown
Contributor

@yangxk1 yangxk1 left a comment

Choose a reason for hiding this comment

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

LGTM, thank you~ @SYaoJun

@yangxk1 yangxk1 merged commit 481c6f9 into apache:main Feb 24, 2026
12 checks 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.

4 participants