Deduplicate parsing
Los mantenedores suelen responder en 10 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 55/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- rust
- Área
- backend, performance
Línea de trabajo
Start by tracing execute_with_reader(), validate(), and prepare_data_with_reader() to see where each builds the source tree and AST. Work out how those parsed values could be reused across the calls; done means the same query is parsed once while dispatch, validation, and data preparation still work correctly.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
This issue is an extract from https://github.com/posit-dev/ggsql/pull/558#issuecomment-6597261929
We parse and build the AST several times. If we look at execute_with_reader():
- It builds the source tree, builds the AST for dispatch purposes and calls
validate() validate()itself also rebuilds the source tree and the AST internally (for the 2nd time)execute_with_reader()passes the query onprepare_data_with_reader(), not the source tree / AST that were already build. It also does parsing and AST building (for the 3rd time).
- Lenguaje dominante
- Rust
- Estrellas
- 572
- Forks
- 31
- Merge medio
- 9 d 4 h
- PR fusionados (30 d)
- 12
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: 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 posit-dev/ggsql
-
Missing `linewidth` and `linetype` aesthetics on histogram and bar layersPosiblemente ocupada @teunbrand la tomó hace 11 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 61/100
Los mantenedores suelen responder en 10 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 10 días
-
writer
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 10 días
-
plot building
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 10 días
-
Mention DuckDB extensionPosiblemente ocupada @Anant-mishra1729 la tomó hace 133 días. Abiertodocumentation integration
Dificultad 1/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 10 días
Todos los issues de posit-dev/ggsql
Issues similares
-
area:frontend kind:bug priority:p2 ready
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
bug priority: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
storytold/vectorcraft#6143 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
rubys/roundhouse#1917 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día