Skip to content

chore: Align parser pin for namespace DESCRIBE AST - #185

Merged
osipovartem merged 1 commit into
df55-physical-udf-instancefrom
fix/sqlparser-describe-object-pin-20261113
Nov 13, 2026
Merged

osipovartem merged 1 commit into
df55-physical-udf-instancefrom
fix/sqlparser-describe-object-pin-20261113

Conversation

@osipovartem

@osipovartem osipovartem commented Nov 13, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Align this runtime branch's sqlparser dependency with the merged parser backport #48.

Only three pin lines change: the workspace dependency and the two lockfile source entries. Parser features and DataFusion execution code are unchanged. This prevents two incompatible copies of sqlparser::ast when Rustice enables native DESCRIBE DATABASE/SCHEMA.

The parser backport preserves this fork's legacy VIEW AST and rejects swallowed EXPLAIN modifiers. Its source is attributed to upstream sqlparser apache#2767, with no duplicate Apache contribution.

Validation

  • Parser: 1,590 passed / 0 failed / 7 ignored; strict all-target/all-feature Clippy, serde-only common-test compilation and formatting passed.
  • Parser independent review: approved.
  • This manifest-only diff: git diff --check passed.
  • Locked offline downstream graph: one sqlparser/derive source and one coordinated DataFusion source, passed.
  • Rustice downstream native qualification at local commit f1d42f1012156f7207635cc91ceac0170892bc24: 4,340 passed, zero failed, 187 existing ignored across the executor, functions, catalog, REST, catalog-metadata and role suites. Includes Service-v2 namespace metadata, rollback preservation, quoting and privilege-revocation controls.
  • Independent read-only source review: approved, no blockers in the manifest/lockfile change.
  • The intermediate draft Rust CI run was intentionally canceled to avoid duplicating local builds. This is downstream qualification, not a claim that the entire DataFusion workspace suite or remote CI passed.

Rustice stock-connector and strict Bronze follow-ups remain tracked in Rustice #1256; no wire or full-live-audit claim is made by this pin-only PR.

Base is df55-physical-udf-instance, the actual runtime revision used by Rustice, not the diverged default branch.

@osipovartem
osipovartem marked this pull request as ready for review November 13, 2026 10:23
@osipovartem
osipovartem merged commit 1eba506 into df55-physical-udf-instance Nov 13, 2026
33 of 36 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.

1 participant