API change: Column methods should throw exceptions consistently
Maintainer antworten meist innerhalb von 1 Tag
@Enmk arbeitet bereits daran.
Seit 24.11.2022.
Bewertung
Dieses Issue wurde noch nicht bewertet.
Beschreibung
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.
- Vorherrschende Sprache
- C
- Sterne
- 386
- Forks
- 212
- Ø Merge
- 18 Std. 22 Min.
- Gemergte PRs (30 T.)
- 4
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus ClickHouse/clickhouse-cpp
-
enhancement pg_clickhouse
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
ClickHouse/clickhouse-cpp#478 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Unknown type names inside a container are misread instead of rejectedEvtl. vergeben @Hakaou hat das vor 3 Tagen übernommen. Offen
Schwierigkeit 4/5 1-2 Tage Anfängerfreundlichkeit 48/100
ClickHouse/clickhouse-cpp#589 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Array(Nested(...)) columns silently decode as Array(Nothing) and desynchronize the native block streamEvtl. vergeben @Hakaou hat das vor 4 Tagen übernommen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 68/100
ClickHouse/clickhouse-cpp#571 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 62/100
ClickHouse/clickhouse-cpp#565 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Client::Insert concatenates the raw table name into SQL — table names needing backquotes cannot be usedEvtl. wieder frei Ein Pull Request dazu wurde ohne Merge geschlossen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100
ClickHouse/clickhouse-cpp#560 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in ClickHouse/clickhouse-cpp
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
valkey-io/valkey#4979 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
colbymchenry/codegraph#2770 ·
Maintainer antworten meist innerhalb von 1 Tag
-
area/ysql kind/enhancement priority/medium status/awaiting-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 73/100
yugabyte/yugabyte-db#35578 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
Maintainer antworten meist innerhalb von 1 Tag
-
test_announce_and_connect_via_dns binds a fixed port that other tests' nodes can be handedEvtl. vergeben @ksedgwic hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 83/100
ElementsProject/lightning#9642 ·
Maintainer antworten meist innerhalb von 2 Tagen