ClickHouse / ClickHouse/clickhouse-java
Support all Aggregate Functions as parameter for columns of type AggregateFunction
- Dominant language
- Java
- Stars
- 1.6k
- Forks
- 636
- Avg merge
- 2d 23h
- Merged PRs (30d)
- 29
Description
Currently, only [groupBitmap](https://clickhouse.com/docs/en/sql-reference/aggregate-functions/reference/groupbitmap) is [supported](https://github.com/ClickHouse/clickhouse-java/blob/d175cd7d0e8f0427a8e2bc1b20c597990cd507b4/clickhouse-data/src/main/java/com/clickhouse/data/format/ClickHouseRowBinaryProcessor.java#L510-L515) as parameter for a table column of type [AggregateFunction](https://clickhouse.com/docs/en/sql-reference/data-types/aggregatefunction/).
It would be good if [all](https://clickhouse.com/docs/en/sql-reference/aggregate-functions/reference/) aggregate functions are supported.
Contributor guide
Research direction
Inspect ClickHouseRowBinaryProcessor.java around lines 510-515 and review the AggregateFunction and aggregate-function reference documentation. Check merged pull request #1345 and the current implementation first; done means supported AggregateFunction column parameters cover the requested aggregate functions without breaking existing handling.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- databases
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100