apache / apache/datafusion-sqlparser-rs
Remove or document second field in `Value::Number`
オープン
- 主要言語
- Rust
- スター
- 3.5k
- フォーク
- 772
- 平均マージ
- 4日 9時間
- マージ済み PR(30日)
- 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 を見つけます。long フィールドが必要かどうかを判断するため、サポートされている SQL ダイアレクトを確認します。完了条件は、バリアントが自己文書化されているかフィールドが削除され、すべての参照がコンパイルでき、既存のテストがパスすることです。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- rust, sql
- 領域
- databases
- issue の種類
- リファクタリング
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 45/100