apache / apache/datafusion-sqlparser-rs

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

未關閉
#363 0 則留言 2 個 reaction 已指派 0 人 在 GitHub 檢視
主要語言
Rust
星號
3.5k
分支
774
平均合併
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 摘要。