Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

Split `src/parser/mod.rs` and `src/ast/mod.rs` into smaller modules

Offen
  • Stimmt etwas nicht?
#2,591 5 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 2 Tagen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Anfängerfreundlichkeit
38/100
Issue-Typ
Refactoring
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
rust, sql
Bereich
build-system, compilers

Rechercherichtung

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.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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:

  1. Do this operation in single, reasonably sized PRs
  2. 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.

Vorherrschende Sprache
Rust
Sterne
3.5k
Forks
774
Ø Merge
3 T. 23 Std.
Gemergte PRs (30 T.)
53

Entwicklungsumgebung

Die Einrichtungsdateien dieses Projekts haben wir noch nicht geprüft. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus apache/datafusion-sqlparser-rs

Alle Issues in apache/datafusion-sqlparser-rs

Ähnliche Issues

Weitere Issues zu Rust

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.