apache / apache/datafusion-sqlparser-rs

Remove or document second field in `Value::Number`

未关闭
#363 0 条评论 2 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
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

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。