4paradigm / 4paradigm/OpenMLDB
Category aggregation functions
- Dominant language
- C++
- Stars
- 1.7k
- Forks
- 331
- Avg merge
- 12d 12h
- Merged PRs (30d)
- 1
Description
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_
Contributor guide
Research direction
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.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- sql
- Domain
- backend, databases
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100