Redshift dialect: support nested block comments

Ouverte
  • Quelque chose ne va pas ?
#2,959 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Cette issue n'a pas encore été évaluée.

Description

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.

Langage dominant
Rust
Étoiles
3.5k
Forks
805
Merge moyen
2 j 3 h
PR mergées (30 j)
116

Préparer son environnement

  • Aucun Dockerfile ni fichier Docker Compose
  • Propose un modèle de pull request
  • Aucun guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de apache/datafusion-sqlparser-rs

Toutes les issues de apache/datafusion-sqlparser-rs

Issues similaires

Plus d'issues Rust

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.