Split `src/parser/mod.rs` and `src/ast/mod.rs` into smaller modules
維護者通常 2 天內回覆
還沒有人認領這個 Issue。
評估
- 難度
- 5/5
- 預估耗時
- 一週以上
- 新手友好度
- 38/100
- Issue 類型
- 重構
- 描述清晰度
- 基本清楚
- 活躍度
- 活躍
研究方向
Start by reading src/parser/mod.rs and src/ast/mod.rs, then review the discussion in #944 and #1581 for the agreed boundaries and sequencing. The work should be divided into small pure-move PRs, with a CI script checking that each move is complete and contains no unintended changes; completion means both large modules are split without behavioral changes.
由索引模型根據 Issue 內容生成。
描述
Several source files have grown past the point where navigating them is practical, with the parser mod at over 20k and the ast mod at over 10k.
This has come up before. #944 (2023, still open) proposed one parser file per statement, and #1581 (2024) split the whole parser in one draft PR. Both got agreement on the goal, conditioned on a series of small pure-move PRs. The earlier one-shot attempts #344 and #351 (2021) stalled on reviewer bandwidth, with concerns about conflicts with open PRs and about coupled code being easier to read in one file.
I suggest we try to do this before the next release, currently roughly planned for end of October (https://github.com/apache/datafusion-sqlparser-rs/issues/2454), as a series of pure move PRs. I believe that, given the size of the task, it may be desirable to:
- Do this operation in single, reasonably sized PRs
- Prepare BEFORE a script for the CI that checks that the move is actually a move, and it is complete
If agreed, this supersedes #944.
- 主要語言
- Rust
- 星號
- 3.5k
- 分支
- 774
- 平均合併
- 4 天 23 分鐘
- 30 天內合併 PR
- 55
環境準備
我們還沒有檢查這個專案的環境設定檔。先看它的 README，通用步驟見我們的新手貢獻指南。
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
apache/datafusion-sqlparser-rs 的其他 Issue
-
Hive
難度 2/5 1-3 小時 新手友好度 70/100
apache/datafusion-sqlparser-rs#2589 ·
維護者通常 2 天內回覆
-
難度 2/5 1-3 小時 新手友好度 78/100
apache/datafusion-sqlparser-rs#2461 · 2 則留言 ·
維護者通常 2 天內回覆
-
難度 2/5 1-3 小時 新手友好度 73/100
apache/datafusion-sqlparser-rs#2407 ·
維護者通常 2 天內回覆
-
難度 2/5 1-3 小時 新手友好度 65/100
apache/datafusion-sqlparser-rs#2393 ·
維護者通常 2 天內回覆
-
難度 1/5 1 小時以內 新手友好度 88/100
apache/datafusion-sqlparser-rs#2390 · 1 則留言 ·
維護者通常 2 天內回覆
查看 apache/datafusion-sqlparser-rs 的全部 Issue
相似的 Issue
-
backend::vllm diffusion multimodal
難度 2/5 1-3 小時 新手友好度 72/100
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 85/100
lambdaclass/ethrex#7329 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 78/100
維護者通常 1 天內回覆
-
難度 1/5 1 小時以內 新手友好度 88/100
shadowsocks/shadowsocks-rust#2186 · 1 則留言 ·
-
C-bug S-awaiting-triage
難度 1/5 1 小時以內 新手友好度 92/100
juspay/hyperswitch#14479 ·
維護者通常 1 天內回覆