API change: Column methods should throw exceptions consistently
Les mainteneurs répondent en général sous 1 jour
@Enmk y travaille déjà.
Depuis le 24/11/2022.
Évaluation
Cette issue n'a pas encore été évaluée.
Description
There are multiple places throughout the library that are really relaxed about input values\types, like:
- overrides of
Column::Append(ColumnRef column)that just do nothing if column is of not expected type - overrides of
Column::Loadthat return false if column failed to load, sometimes eating an exception that could point to the root cause. - etc.
Fixes scheduled for next API-breaking release (3.0?):
Column::Appendoverrides should throw an exception if the input column is of the wrong type.Column::Loadshould change return type tovoidand throw an exception on error.
- Langage dominant
- C
- Étoiles
- 385
- Forks
- 211
- Merge moyen
- 18 h 22 min
- PR mergées (30 j)
- 4
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de ClickHouse/clickhouse-cpp
-
enhancement pg_clickhouse
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
ClickHouse/clickhouse-cpp#478 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Unknown type names inside a container decode silently as wrong dataPeut-être pris @Hakaou l’a pris il y a 1 jour. Ouverte
Difficulté 4/5 1-2 jours Accessibilité débutants 48/100
ClickHouse/clickhouse-cpp#589 ·
Les mainteneurs répondent en général sous 1 jour
-
Array(Nested(...)) columns silently decode as Array(Nothing) and desynchronize the native block streamPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 68/100
ClickHouse/clickhouse-cpp#571 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 62/100
ClickHouse/clickhouse-cpp#565 ·
Les mainteneurs répondent en général sous 1 jour
-
Client::Insert concatenates the raw table name into SQL — table names needing backquotes cannot be usedPeut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
ClickHouse/clickhouse-cpp#560 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de ClickHouse/clickhouse-cpp
Issues similaires
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
EchoTools/nevr-runtime#698 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
FujiNetWIFI/fujinet-firmware#2079 ·
Les mainteneurs répondent en général sous 1 jour
-
bug Official Image release/25.12 Supported Device target/kirkwood
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
area/archive bug
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
DarkFlippers/qUnleashed#374 ·
Les mainteneurs répondent en général sous 1 jour
-
Missing options in yara man pageOuvertebug
Difficulté 1/5 Moins d'une heure Accessibilité débutants 84/100
VirusTotal/yara#2255 ·