Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove do content as propriedades parent_title, parent_slug e parent_username #677

Merged
merged 2 commits into from
Aug 25, 2022

Conversation

filipedeschamps
Copy link
Owner

Este PR traz a breaking change comentada nesta publicação e também remove o ruído das queries causado por as redundantes.

E foi delicinha de fazer só por conta do watch nos testes automatizados. Eu nunca dormiria tranquilo com uma refatoração dessas se não tivéssemos os atuais testes 🤝

@vercel
Copy link

vercel bot commented Aug 24, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
tabnews ✅ Ready (Inspect) Visit Preview Aug 25, 2022 at 3:03PM (UTC)

@filipedeschamps
Copy link
Owner Author

Resultados

Eu não sei se estou testando algo errado, mas fiquei surpreso com os resultados.

Antes: https://tabnews-git-parent-root-content-tabnews.vercel.app/api/v1/contents?strategy=relevant
Depois: https://tabnews-git-content-parent-properties-tabnews.vercel.app/api/v1/contents?strategy=relevant

Antes Depois
image image

Estou me perguntando aqui: pelo fato dos conteúdos root não terem parent, será que o scan que o banco tinha que fazer antes era de toda a tabela content e toda tabela user a procura de um match que nunca iria existir?

@filipedeschamps filipedeschamps force-pushed the content-parent-properties branch from e66a311 to 3b2e04b Compare August 25, 2022 15:01
@filipedeschamps filipedeschamps merged commit 8185e1c into main Aug 25, 2022
@filipedeschamps filipedeschamps deleted the content-parent-properties branch August 25, 2022 15:04
@filipedeschamps
Copy link
Owner Author

Merged!! Let's gooooo!!

Resultados no ambiente de Produção

Não foi o mesmo salto que teve no ambiente de Homologação, mas depois eu vou tentar montar um gráfico para ver se nele dá para visualizar algum impacto geral 🤝

Antes Depois
image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant