ClickHouse / ClickHouse/clickhouse-cpp
API change: Column methods should throw exceptions consistently
- 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.