Redshift dialect: support nested block comments

Aberta
  • Algo errado?
#2,959 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Esta issue ainda não foi avaliada.

Descrição

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.

Linguagem predominante
Rust
Estrelas
3.5k
Forks
805
Merge médio
2d 3h
PRs com merge (30d)
116

Preparar o ambiente

  • Sem Dockerfile nem arquivo Docker Compose
  • Tem um modelo de pull request
  • Sem guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de apache/datafusion-sqlparser-rs

Todas as issues de apache/datafusion-sqlparser-rs

Issues semelhantes

Mais issues de Rust

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.