Skip to content

Core: Align content stats fields with latest Spec changes#16439

Merged
danielcweeks merged 4 commits into
apache:mainfrom
nastra:content-stats-align-impl
Jun 8, 2026
Merged

Core: Align content stats fields with latest Spec changes#16439
danielcweeks merged 4 commits into
apache:mainfrom
nastra:content-stats-align-impl

Conversation

@nastra

@nastra nastra commented May 20, 2026

Copy link
Copy Markdown
Contributor

This aligns the different content stats field with the latest spec changes that were introduced by #14234. It also updates how the content stats schema is generated for Geo & Variant.
This PR doesn't cover the updated calculations for the two reserved metadata fields and will be handled in a separate PR.

The PR was created with the help of Claude Opus 4.7 and manually adjusted in a few places.

@github-actions github-actions Bot added the core label May 20, 2026
@nastra nastra force-pushed the content-stats-align-impl branch 3 times, most recently from 355684d to 51be71e Compare May 20, 2026 13:00
@nastra nastra requested a review from danielcweeks May 27, 2026 14:13
Comment thread core/src/main/java/org/apache/iceberg/StatsUtil.java Outdated
@nastra nastra force-pushed the content-stats-align-impl branch from 51be71e to 7165fb6 Compare June 8, 2026 08:50
@nastra nastra force-pushed the content-stats-align-impl branch from 7165fb6 to 88bd2ac Compare June 8, 2026 10:00

@danielcweeks danielcweeks 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.

Verified all the updated ids/mappings per the spec. Looks good to me.

@danielcweeks danielcweeks merged commit d2de290 into apache:main Jun 8, 2026
53 checks passed
@nastra nastra deleted the content-stats-align-impl branch June 9, 2026 05:19
@nssalian nssalian added this to the Iceberg 1.12.0 milestone Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants