ClickHouse / ClickHouse/clickhouse-cpp

API change: Column methods should throw exceptions consistently

オープン
#243 コメント 1 件 リアクション 0 件 担当者 1 名 @Enmk が担当を希望しています GitHub で見る
3.0
主要言語
C
スター
382
フォーク
209
平均マージ
1日 22時間
マージ済み PR(30日)
13

説明

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.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。