ClickHouse / ClickHouse/clickhouse-cpp

API change: Column methods should throw exceptions consistently

Offen
#243 1 Kommentar 0 Reaktionen 1 zugewiesene Person Beansprucht von @Enmk Auf GitHub ansehen
3.0
Vorherrschende Sprache
C
Sterne
382
Forks
209
Ø Merge
1 T. 22 Std.
Gemergte PRs (30 T.)
13

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::Load` that 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::Append` overrides should throw an exception if the input column is of the wrong type.
- `Column::Load` should change return type to `void` and throw an exception on error.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.