docs: drop TableSchema upgrade-guide note#62
Merged
adriangb merged 1 commit intoMay 26, 2026
Conversation
The TableSchema::new/from_file_schema deprecations surface via compiler deprecation warnings that point callers to the builder, and nothing is required on upgrade, so an upgrade-guide entry isn't needed (per review).
github-actions
Bot
added
the
documentation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Which issue does this PR close?
N/A — addresses review feedback on apache#22496 (discussion r3306491587).
Rationale for this change
Per review: there is nothing a caller needs to do on upgrade for the
TableSchemabuilder change.TableSchema::new/TableSchema::from_file_schema/ thewith_table_partition_colssetter are all#[deprecated], so the compiler deprecation warnings already point callers at the builder /Fromconversions. An upgrade-guide entry is therefore unnecessary.What changes are included in this PR?
Removes the
### TableSchema gains a builder; table_partition_cols() now returns &Fieldssection fromdocs/source/library-user-guide/upgrading/55.0.0.md(47 lines).The two pre-existing sections (
is_dynamic_physical_exprdeprecation andFilePruner::try_new) come from apache#22460 and are left untouched — after this change the file is byte-identical to its pre-apache#22496 state.Are these changes tested?
No tests needed — documentation-only change.
Are there any user-facing changes?
Removes an upgrade-guide note for an unreleased (55.0.0) change. No code or API changes.
Generated by Claude Code