Redshift dialect: `--` comment should end at a carriage return

Ouverte
  • Quelque chose ne va pas ?
#2,958 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

In the Redshift dialect a -- comment runs to the next \n, but Redshift ends it at a lone \r too, as Postgres does. sqlparser already stops at \r for PostgreSqlDialect (tokenizer tokenize_single_line_comment).

select 1 as a --\r, pg_backend_pid()

(\r is a literal carriage return.)

  • sqlparser 0.64.0: SingleLineComment("\r, pg_backend_pid()"); parses as SELECT 1 AS a.
  • Redshift: two columns, a = 1, pg_backend_pid = 1073905994.

Redshift is based on PostgreSQL 8.0.2 (https://docs.aws.amazon.com/redshift/latest/dg/c_redshift-and-postgres-sql.html), whose lexer treats both \n and \r as newlines. Could RedshiftSqlDialect share the Postgres behavior here?

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.