cube-js/cube

Native distinct counts in ClickHouse dialect

Open

#2 334 ouverte le 11 mars 2021

Voir sur GitHub
 (2 commentaires) (1 réaction) (0 assignés)Rust (1 965 forks)batch import
driver:clickhousegood first issuehelp wanted

Métriques du dépôt

Stars
 (19 563 stars)
Métriques de merge PR
 (Merge moyen 5j 16h) (138 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez l'adaptateur ClickHouseQuery dans cubejs schema compiler/src/adapter/ClickHouseQuery.js et ajoutez la prise en charge des fonctions uniq, uniqExact et uniqHLL12 pour les agrégations count distinct, en suivant le modèle des autres adaptateurs de bases de données.
Stack technique
javascript
Domaine
backenddatabase
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80