apache / apache/datafusion-sqlparser-rs
Cannot parse named windows with modifiers
- Lenguaje dominante
- Rust
- Estrellas
- 3.5k
- Forks
- 772
- Merge medio
- 4 d 9 h
- PR fusionados (30 d)
- 17
Descripción
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
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Línea de trabajo
Comienza examinando cómo sqlparser-rs analiza la expresión named-window en la cláusula OVER; después, revisa el issue #867 y el pull request #881 para consultar el diseño anterior. Añade cobertura de análisis para los ejemplos de MySQL y BigQuery, y verifica que los named windows con modificadores se acepten sin provocar regresiones en los named windows simples.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- rust, sql
- Área
- compilers, databases
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 40/100