Skip to content

Phase 1: partition zoned stats internally - #7940

Closed
gatesn wants to merge 1 commit into
developfrom
ngates/7939-phase-1
Closed

gatesn wants to merge 1 commit into
developfrom
ngates/7939-phase-1

Conversation

@gatesn

@gatesn gatesn commented May 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Implements phase 1 of #7939 by moving fixed-size zone stats partitioning into ZonedStrategy.

  • partitions zone stats internally from arbitrary input chunk boundaries
  • keeps the data child write path independent from stats partitioning
  • preserves the current single-zone-map metadata and reader pruning behavior
  • adds regression coverage for input chunks smaller than, larger than, and unaligned with the configured zone size

Validation

  • cargo +nightly fmt --all
  • cargo test -p vortex-layout layouts::zoned
  • cargo test -p vortex-file test_segment_ordering_zonemaps_after_data
  • cargo check -p vortex-file
  • cargo clippy --all-targets --all-features

Signed-off-by: "Nicholas Gates" <[email protected]>
@gatesn gatesn mentioned this pull request May 15, 2026
25 tasks
@github-actions

Copy link
Copy Markdown
Contributor

This PR has been marked as stale because it has been open for 14 days with no activity. Please comment or remove the stale label if you wish to keep it active, otherwise it will be closed in 7 days

@github-actions github-actions Bot added the stale This PR is stale and will be auto-closed soon label May 31, 2026
@github-actions

github-actions Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

This PR was closed because it has been inactive for 7 days since being marked as stale.

@github-actions github-actions Bot closed this Jun 7, 2026
@robert3005
robert3005 deleted the ngates/7939-phase-1 branch October 16, 2026 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale This PR is stale and will be auto-closed soon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant