apache / apache/datafusion-sqlparser-rs

feat: Support IGNORE NULLS only for allowed builtin window functions

Offen
#1,207 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.