Repository navigation
chore: synchronize schema-options parser pin - #187
Merged
osipovartem merged 1 commit intoNov 13, 2026
Merged
osipovartem merged 1 commit into
osipovartem merged 1 commit into
Conversation
osipovartem
merged commit Nov 13, 2026
dbbcfe5
into
df55-physical-udf-instance
34 of 35 checks passed
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
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?
No separate issue. Coordinated parser pin for Rustice's managed CREATE SCHEMA metadata workflow.
Rationale for this change
DataFusion and downstream Rustice must share one sqlparser AST identity when retaining Snowflake schema properties.
What changes are included in this PR?
Update only the parser revision in Cargo.toml and the two matching lockfile entries to
f3685b358411ba43f0f9b6fa7f50ed9de1b2b541. No DataFusion operator or planner code changes.Are these changes tested?
The parser passes 1,599 tests and strict all-target/all-feature Clippy. Coordinated downstream builds, 4,354 owning tests, 36 SQLLogicTest unit tests, the unchanged 4,629-case Bronze baseline, and ordinary-daemon stock-connector acceptance pass. Independent source/pin review approved the exact revision.
This is downstream integration evidence, not a claim that the complete standalone DataFusion workspace Clippy/test matrix was rerun. Downstream configured all-targets Clippy and formatting also passed.
Are there any user-facing changes?
The parser retains COMMENT/retention assignments for an execution consumer. This pin does not add standalone DataFusion catalog-property execution.