sqlrs icon indicating copy to clipboard operation
sqlrs copied to clipboard

support count aggregation function

Open Fedomn opened this issue 2 years ago • 2 comments

We already have a simple aggregation implementation in V1 https://github.com/Fedomn/sqlrs/blob/main/src/executor/aggregate/simple_agg.rs, that could be a reference. A different point is that this function need migrate into V2 planner, that could refer to https://github.com/Fedomn/sqlrs/blob/main/src/planner_v2/operator/logical_limit.rs and https://github.com/duckdb/duckdb/blob/main/src/planner/operator/logical_aggregate.cpp

Fedomn avatar Jan 07 '23 13:01 Fedomn

Hi, I want to try it.

caicancai avatar Nov 15 '23 15:11 caicancai

Recently, I've been learning about Risinglight, and I have a certain understanding of SQL, and if I want to try it, it may take some time. Because it's a bit difficult for me

caicancai avatar Nov 15 '23 15:11 caicancai