Skip to content

fix(datafusion): preserve quoted names in temporary views - #1333

Merged
JingsongLi merged 1 commit into
apache:mainfrom
slfan1989:paimon-rust-1332
Nov 6, 2026
Merged

JingsongLi merged 1 commit into
apache:mainfrom
slfan1989:paimon-rust-1332

Conversation

@slfan1989

Copy link
Copy Markdown
Contributor

Purpose

Linked issue: close #1332

Fix CREATE TEMPORARY VIEW losing quoted identifier information, causing views with mixed-case or special-character names to become inaccessible.

Brief change log

  • Use DataFusion's identifier conversion to construct the TableReference.
  • Reuse the same TableReference for registration and IF NOT EXISTS checks.
  • Preserve identifier case and names containing dots, spaces, or embedded double quotes.

Tests

  • Confirmed the new regression test fails before the fix.
  • Added coverage for bare, database-qualified, and fully qualified names, including IF NOT EXISTS and DROP.

API and Format

No.

Documentation

No.

@QuakeWang QuakeWang left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@JingsongLi
JingsongLi merged commit 979f24e into apache:main Nov 6, 2026
15 checks passed
@slfan1989

Copy link
Copy Markdown
Contributor Author

@QuakeWang @JingsongLi Thanks for reviewing and merging this PR!

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.

[Bug] CREATE TEMPORARY VIEW loses quoted identifier information

3 participants