API change: Column methods should throw exceptions consistently
@Enmk 已经在做这个了。
开始于 2022年11月24日。
评估
这个 Issue 还没有评估数据。
描述
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.
- 主要语言
- C
- 星标
- 383
- 派生
- 209
- 平均合并
- 2 天 13 小时
- 30 天内合并 PR
- 7
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
ClickHouse/clickhouse-cpp 的其他 Issue
-
enhancement pg_clickhouse
难度 2/5 1-3 小时 新手友好度 65/100
ClickHouse/clickhouse-cpp#478 · 1 条评论 ·
-
难度 4/5 3-5 天 新手友好度 68/100
ClickHouse/clickhouse-cpp#571 ·
-
难度 4/5 3-5 天 新手友好度 62/100
ClickHouse/clickhouse-cpp#565 ·
-
难度 3/5 1-2 天 新手友好度 72/100
ClickHouse/clickhouse-cpp#560 · 1 条评论 ·
-
难度 4/5 3-5 天 新手友好度 68/100
ClickHouse/clickhouse-cpp#556 · 1 个 reaction ·
查看 ClickHouse/clickhouse-cpp 的全部 Issue
相似的 Issue
-
bug
难度 2/5 1-3 小时 新手友好度 86/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
-
难度 2/5 1-3 小时 新手友好度 78/100
-
难度 2/5 1-3 小时 新手友好度 80/100
es-ude/OnDeviceTraining#479 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 86/100
Ryan-Millard/Img2Num#681 · 2 条评论 ·
维护者通常 2 天内回复