Fedomn / Fedomn/sqlrs

support count aggregation function

Open
#152 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
114
Forks
11
PR merge metrics
No merged PRs in 30d

Description

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

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.