Redshift dialect: support nested block comments

Abierto
  • ¿Algo no está bien?
#2,959 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Este issue todavía no se ha evaluado.

Descripción

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.

Lenguaje dominante
Rust
Estrellas
3.5k
Forks
805
Merge medio
2 d 3 h
PR fusionados (30 d)
116

Preparar el entorno

  • Sin Dockerfile ni archivo de Docker Compose
  • Tiene una plantilla de pull request
  • Sin guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de apache/datafusion-sqlparser-rs

Todos los issues de apache/datafusion-sqlparser-rs

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.