cube-js/cube

Native distinct counts in ClickHouse dialect

Open

#2.334 geöffnet am 11. März 2021

Auf GitHub ansehen
 (2 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Rust (1.965 Forks)batch import
driver:clickhousegood first issuehelp wanted

Repository-Metriken

Stars
 (19.563 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 5T 16h) (138 gemergte PRs in 30 T)

Beschreibung

Is your feature request related to a problem? Please describe. ClickHouse supports a number of functions to count distinct values: uniq, uniqExact, uniqHLL12. However, they are not used in ClickHouseQuery adapter to support countDistinctApprox and other types of aggregations.

Describe the solution you'd like With the guidance of the Cube.js team and taking other adapters as examples, these functions should be used in the adapter.

Describe alternatives you've considered —

Additional context Without this feature, countDistinctApprox is not supported at all, and — I think — countDistinct works slower than it could.

Contributor Guide

Research-Richtung
Untersuche den ClickHouseQuery Adapter in cubejs schema compiler/src/adapter/ClickHouseQuery.js und füge Unterstützung für die Funktionen uniq, uniqExact und uniqHLL12 für count distinct Aggregationen hinzu, dem Muster anderer Datenbankadapter folgend.
Tech Stack
javascript
Domain
backenddatabase
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80