Split `src/parser/mod.rs` and `src/ast/mod.rs` into smaller modules
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 38/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Área
- build-system, compilers
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Rust
- Estrellas
- 3.5k
- Forks
- 774
- Merge medio
- 3 d 23 h
- PR fusionados (30 d)
- 54
Preparar el entorno
Aún no hemos revisado los archivos de configuración de este proyecto. Empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de apache/datafusion-sqlparser-rs
-
Hive
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
apache/datafusion-sqlparser-rs#2589 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
apache/datafusion-sqlparser-rs#2461 · 2 comentarios ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
apache/datafusion-sqlparser-rs#2407 ·
Los mantenedores suelen responder en 2 días
-
Add SQL/PGQ keywordsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
apache/datafusion-sqlparser-rs#2393 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
apache/datafusion-sqlparser-rs#2390 · 1 comentario ·
Los mantenedores suelen responder en 2 días
Todos los issues de apache/datafusion-sqlparser-rs
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
vercel-labs/agent-browser#2017 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
tursodatabase/turso#9405 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
PolyMeilex/Neothesia#447 ·
Los mantenedores suelen responder en 1 día
-
backend::vllm diffusion multimodal
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
trezor/trezor-firmware#7985 ·
Los mantenedores suelen responder en 2 días