apache / apache/datafusion-sqlparser-rs
Cannot parse named windows with modifiers
- Linguagem predominante
- Rust
- Estrelas
- 3.5k
- Forks
- 772
- Merge médio
- 4d 9h
- PRs com merge (30d)
- 17
Descrição
sqlparser-rs only supports named windows that are an identifier. This is fine for queries like:
```sql
SELECT sum(salary) OVER w, avg(salary) OVER w
FROM empsalary
WINDOW w AS (PARTITION BY depname ORDER BY salary DESC);
```
However, it is insufficient for queries in at least BigQuery and MySQL where modifiers can be added to the named window in the OVER clause.
[MySQL named window example](https://dev.mysql.com/doc/refman/8.0/en/window-functions-named-windows.html)
```sql
SELECT
DISTINCT year, country,
FIRST_VALUE(year) OVER (w ORDER BY year ASC) AS first,
FIRST_VALUE(year) OVER (w ORDER BY year DESC) AS last
FROM sales
WINDOW w AS (PARTITION BY country)
```
[BigQuery named window example](https://cloud.google.com/bigquery/docs/reference/standard-sql/window-function-calls)
```sql
SELECT item, purchases, category, LAST_VALUE(item)
OVER (
item_window
ROWS BETWEEN 1 PRECEDING AND 1 FOLLOWING
) AS most_popular
FROM Produce
WINDOW item_window AS (
PARTITION BY category
ORDER BY purchases)
```
This would build upon prior to work:
- https://github.com/sqlparser-rs/sqlparser-rs/issues/867
- https://github.com/sqlparser-rs/sqlparser-rs/pull/881
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Direção de pesquisa
Comece examinando como o sqlparser-rs faz o parsing da expressão named-window na cláusula OVER e, em seguida, revise a issue #867 e o pull request #881 para consultar o design anterior. Adicione cobertura de parsing para os exemplos de MySQL e BigQuery e verifique se named windows com modificadores são aceitas sem causar regressões em named windows simples.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- rust, sql
- Domínio
- compilers, databases
- Tipo de issue
- Funcionalidade
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 40/100