Redshift dialect: support nested block comments

Offen
  • Stimmt etwas nicht?
#2,959 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Beschreibung

Redshift

Redshift nests /* */ comments, as Postgres does, but RedshiftSqlDialect doesn't set supports_nested_comments, so sqlparser ends the comment at the first */.

select 1 /* /* */ , ' */ , pg_backend_pid() -- '
  • sqlparser 0.64.0: MultiLineComment(" /* "), then a string; parses as SELECT 1, ' */ , pg_backend_pid() -- '.
  • Redshift: two columns, 1 and 1073905994, so the comment ran to the second */.

The overlapping form select 1 /*/*/ , ' */ */ , pg_backend_pid() -- ' behaves the same way on both sides.

Postgres documents nesting in "Comments" (https://www.postgresql.org/docs/8.0/sql-syntax.html#SQL-SYNTAX-COMMENTS), and Redshift is based on PostgreSQL 8.0.2.

Vorherrschende Sprache
Rust
Sterne
3.5k
Forks
805
Ø Merge
2 T. 3 Std.
Gemergte PRs (30 T.)
116

Entwicklungsumgebung

  • Kein Dockerfile und keine Docker-Compose-Datei
  • Hat eine Pull-Request-Vorlage
  • Kein Beitragsleitfaden

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus apache/datafusion-sqlparser-rs

Alle Issues in apache/datafusion-sqlparser-rs

Ähnliche Issues

Weitere Issues zu Rust

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.