4paradigm / 4paradigm/OpenMLDB

Category aggregation functions

Open
#3,014 2 comments 0 reactions 1 assignee Claimed by @aceforeverd View on GitHub
enhancement execute-engine
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.