Skip to content

fix(table): enable global index by default#587

Merged
JingsongLi merged 2 commits into
apache:mainfrom
XiaoHongbo-Hope:fix_global_index_enabled_default
Jul 22, 2026
Merged

fix(table): enable global index by default#587
JingsongLi merged 2 commits into
apache:mainfrom
XiaoHongbo-Hope:fix_global_index_enabled_default

Conversation

@XiaoHongbo-Hope

@XiaoHongbo-Hope XiaoHongbo-Hope commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Align the Rust global-index.enabled default with Java Paimon and PyPaimon. When the option is absent, Rust scans can now use existing global indexes; an explicit false still disables them.

Changes

  • Default global-index.enabled to true when it is not configured.
  • Cover the default and explicit true / false overrides at the option layer.
  • Exercise the regular table-scan path without an explicit override and verify that it reads the committed BTree index row ranges.
  • Update the SQL option reference with the corrected default.

Testing

  • cargo test --locked -p paimon --lib (1800 passed, 1 ignored)
  • cargo test --locked -p paimon --lib table::btree_global_index_build_builder::tests (23 passed)
  • Parent/head regression check: parent returns no row ranges; this PR returns [0,0] and [2,2]
  • cargo fmt --all -- --check
  • cargo clippy --locked -p paimon --all-targets -- -D warnings

@QuakeWang QuakeWang left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

@JingsongLi
JingsongLi merged commit 2b8f908 into apache:main Jul 22, 2026
12 of 13 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