apache / apache/datafusion-sqlparser-rs
Remove or document second field in `Value::Number`
未关闭
- 主要语言
- Rust
- 星标
- 3.5k
- 派生
- 772
- 平均合并
- 4 天 9 小时
- 30 天内合并 PR
- 17
描述
Value::Number should be changed from a tuple variant to a struct variant so that the field names are self documenting.
Something like: `Number { value: String, long: bool }`
Possibly the long field could be just removed instead but not im not very familar with various SQL languages.
贡献指南
这个仓库没有索引到贡献指南
调研方向
首先定位 Value::Number 的定义,以及使用它的每个构造器和 pattern match。检查支持的 SQL 方言,以确定 long 字段是否必需;完成的标准是该变体能够自我说明或移除该字段,所有引用都能编译,并且现有测试通过。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- rust, sql
- 领域
- databases
- Issue 类型
- 重构
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 45/100