Skip to content

Only support escape literals for Postgres, Redshift and generic dialect #1674

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Jan 24, 2025

Conversation

hansott
Copy link
Contributor

@hansott hansott commented Jan 21, 2025

Example in MySQL:

Screenshot 2025-01-21 at 18 13 08

Copy link
Contributor

@iffyio iffyio left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @hansott!
cc @alamb

@iffyio
Copy link
Contributor

iffyio commented Jan 23, 2025

@hansott when you get the chance if you could take a look at the conflicts in the branch

@hansott
Copy link
Contributor Author

hansott commented Jan 23, 2025

Yup, I'm on it

…o escape-literals

* 'main' of github.com:hansott/datafusion-sqlparser-rs:
  National strings: check if dialect supports backslash escape (apache#1672)
  Add support for Create Iceberg Table statement for Snowflake parser (apache#1664)
  Add support for Snowflake account privileges (apache#1666)
  Update rat_exclude_file.txt (apache#1670)
  Update verson to 0.54.0 and update changelog (apache#1668)
  Add support for Snowflake AT/BEFORE (apache#1667)
  Add support for qualified column names in JOIN ... USING (apache#1663)
  Add support for `IS [NOT] [form] NORMALIZED` (apache#1655)
  fix parsing of `INSERT INTO ... SELECT ... RETURNING ` (apache#1661)
  Add support for Snowflake column aliases that use SQL keywords (apache#1632)
@hansott
Copy link
Contributor Author

hansott commented Jan 23, 2025

@iffyio done! Also adapted the test slightly.

@hansott
Copy link
Contributor Author

hansott commented Jan 23, 2025

Fixed lint error!

@iffyio iffyio merged commit ef072be into apache:main Jan 24, 2025
9 checks passed
@hansott hansott deleted the escape-literals branch January 24, 2025 09:00
@alamb
Copy link
Contributor

alamb commented Jan 24, 2025

Thanks @hansott and @iffyio 🚀

Vedin pushed a commit to Embucket/datafusion-sqlparser-rs that referenced this pull request Feb 3, 2025
Vedin pushed a commit to Embucket/datafusion-sqlparser-rs that referenced this pull request Feb 3, 2025
Vedin added a commit to Embucket/datafusion-sqlparser-rs that referenced this pull request Feb 3, 2025
ayman-sigma pushed a commit to sigmacomputing/sqlparser-rs that referenced this pull request Apr 10, 2025
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.

3 participants