Skip to content

refractor(C++): remove hardcoded row group size, keep 64M default#872

Merged
yangxk1 merged 1 commit intoapache:mainfrom
SYaoJun:220_row_config
Feb 25, 2026
Merged

refractor(C++): remove hardcoded row group size, keep 64M default#872
yangxk1 merged 1 commit intoapache:mainfrom
SYaoJun:220_row_config

Conversation

@SYaoJun
Copy link
Contributor

@SYaoJun SYaoJun commented Feb 20, 2026

Reason for this PR

Currently, row group size is hard code: 64 * 1024 * 1024

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@codecov-commenter
Copy link

codecov-commenter commented Feb 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.22%. Comparing base (3e6560f) to head (58a62ab).
⚠️ Report is 9 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #872      +/-   ##
============================================
+ Coverage     80.21%   80.22%   +0.01%     
  Complexity      615      615              
============================================
  Files            93       93              
  Lines         10258    10261       +3     
  Branches       1050     1050              
============================================
+ Hits           8228     8232       +4     
+ Misses         1790     1789       -1     
  Partials        240      240              
Flag Coverage Δ
cpp 71.54% <100.00%> (+0.03%) ⬆️

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.

@SYaoJun SYaoJun marked this pull request as draft February 23, 2026 05:23
@SYaoJun SYaoJun mentioned this pull request Feb 24, 2026
@SYaoJun SYaoJun marked this pull request as ready for review February 25, 2026 04:04
Copy link
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~

@yangxk1 yangxk1 merged commit fe013bc into apache:main Feb 25, 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.

3 participants