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

Aberta
  • Algo errado?
#2,958 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

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?

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.