Skip to content

refactor(vindex): trim granule planning overhead - #892

Merged
JingsongLi merged 1 commit into
apache:mainfrom
jerry-024:codex/ivf-granule-cleanup
Sep 21, 2026
Merged

JingsongLi merged 1 commit into
apache:mainfrom
jerry-024:codex/ivf-granule-cleanup

Conversation

@jerry-024

Copy link
Copy Markdown
Contributor

Summary

Follow-up cleanup for #800:

  • Return before byte-range aggregation when planning has already selected whole-shard fallback.
  • Remove the synthetic aggregate byte range from the fallback granule; that range was never consumed.
  • Remove unused derives and a no-op test clone, and tie the blocking-pool regression setup to QUEUE_CAPACITY.

This does not change the selected Granule plan or index output. It only skips work on paths that already return the whole-shard plan.

Validation

  • 18 Granule pipeline unit tests
  • 2 Parquet granule tests
  • cargo +1.94.0 clippy -p paimon --lib --tests -- -D warnings
  • git diff --check

@JingsongLi JingsongLi left a comment

Copy link
Copy Markdown
Contributor

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 db380ca into apache:main Sep 21, 2026
14 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.

2 participants