API change: Column methods should throw exceptions consistently
@Enmk ci sta già lavorando.
Dal 24/11/2022.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
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.
- Lingua principale
- C
- Stelle
- 383
- Fork
- 209
- Merge medio
- 2g 13h
- PR unite (30g)
- 7
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di ClickHouse/clickhouse-cpp
-
enhancement pg_clickhouse
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
ClickHouse/clickhouse-cpp#478 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 68/100
ClickHouse/clickhouse-cpp#571 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 62/100
ClickHouse/clickhouse-cpp#565 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
ClickHouse/clickhouse-cpp#560 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 68/100
ClickHouse/clickhouse-cpp#556 · 1 reazione ·
Tutte le issue di ClickHouse/clickhouse-cpp
Issue simili
-
After `require "openssl"`, a top-level `Digest` is `OpenSSL::Digest`, not the `Digest` moduleAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
ExpressLRS/ExpressLRS#3806 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100