4paradigm / 4paradigm/OpenMLDB

Category aggregation functions

Offen
#3,014 2 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @aceforeverd Auf GitHub ansehen
enhancement execute-engine
Vorherrschende Sprache
C++
Sterne
1.7k
Forks
331
Ø Merge
12 T. 12 Std.
Gemergte PRs (30 T.)
1

Beschreibung

For aggregation functions, there should be window functions (window only), as subset of aggregation functions, is not allowed to used without a window frame. E.g `select at(col, 0) from t2` is invalid.

_Originally posted by @aceforeverd in https://github.com/4paradigm/OpenMLDB/issues/3010#issuecomment-1413453073_

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

The issue references a comment from issue #3010 about window aggregation functions. Look for SQL parsing and validation code, likely where aggregation functions are defined and where window frame requirements are enforced. Test the invalid query `select at(col, 0) from t2` to see the current error, then modify validation to reject window-only aggregates without a frame. Check existing tests for aggregation or window functions to understand the expected behavior.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
sql
Bereich
backend, databases
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 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.