apache / apache/datafusion-sqlparser-rs
feat: Support IGNORE NULLS only for allowed builtin window functions
- Vorherrschende Sprache
- Rust
- Sterne
- 3.5k
- Forks
- 772
- Ø Merge
- 4 T. 9 Std.
- Gemergte PRs (30 T.)
- 17
Beschreibung
Currently parser allows to parse queries which IMHO should be failing by unsupported syntax.
Like
SELECT COUNT(*) IGNORE NULLS OVER() FROM (values (1), (null), (2));
The allowed list for IGNORE NULLS can be found https://github.com/ronsavage/SQL/blob/master/sql-2016.ebnf
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Beginne damit nachzuverfolgen, wie der Parser die Syntax IGNORE NULLS verarbeitet, und vergleiche anschließend die akzeptierten Funktionen mit der erlaubten Liste in der verknüpften SQL-2016-Grammatik. Die Änderung ist abgeschlossen, wenn nicht unterstützte Formen wie COUNT(*) IGNORE NULLS abgelehnt werden, während erlaubte integrierte Fensterfunktionen weiterhin geparst werden können.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rust, sql
- Bereich
- compilers, databases
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100