Deduplicate parsing
I maintainer di solito rispondono entro 10 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 55/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- rust
- Ambito
- backend, performance
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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).
- Lingua principale
- Rust
- Stelle
- 572
- Fork
- 31
- Merge medio
- 9g 4h
- PR unite (30g)
- 12
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di posit-dev/ggsql
-
Missing `linewidth` and `linetype` aesthetics on histogram and bar layersForse già presa @teunbrand l’ha presa 11 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 61/100
I maintainer di solito rispondono entro 10 giorni
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
I maintainer di solito rispondono entro 10 giorni
-
writer
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 10 giorni
-
plot building
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 10 giorni
-
Mention DuckDB extensionForse già presa @Anant-mishra1729 l’ha presa 133 giorni fa. Apertadocumentation integration
Difficoltà 1/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 10 giorni
Tutte le issue di posit-dev/ggsql
Issue simili
-
area:frontend kind:bug priority:p2 ready
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
bug priority: medium
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
storytold/vectorcraft#6143 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
rubys/roundhouse#1917 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 2 giorni
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno