Deduplicate parsing
Mantenedores costumam responder em até 10 dias
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 55/100
- Tipo de issue
- Refatoração
- Clareza
- Razoavelmente clara
- Status de atividade
- Ativa
- Stack de tecnologia
- rust
- Domínio
- backend, performance
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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).
- Linguagem predominante
- Rust
- Estrelas
- 572
- Forks
- 31
- Merge médio
- 9d 4h
- PRs com merge (30d)
- 12
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de posit-dev/ggsql
-
Missing `linewidth` and `linetype` aesthetics on histogram and bar layersTalvez já em andamento @teunbrand assumiu há 11 dias. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 61/100
Mantenedores costumam responder em até 10 dias
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
Mantenedores costumam responder em até 10 dias
-
writer
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
Mantenedores costumam responder em até 10 dias
-
plot building
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 10 dias
-
Mention DuckDB extensionTalvez já em andamento @Anant-mishra1729 assumiu há 133 dias. Abertadocumentation integration
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 76/100
Mantenedores costumam responder em até 10 dias
Todas as issues de posit-dev/ggsql
Issues semelhantes
-
area:frontend kind:bug priority:p2 ready
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
Mantenedores costumam responder em até 1 dia
-
bug priority: medium
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
storytold/vectorcraft#6143 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
rubys/roundhouse#1917 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 2 dias
-
good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
Mantenedores costumam responder em até 1 dia