Diminui idas ao banco de dados na criação e edição de conteúdos, e na edição de usuários #1729
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When creating a content, avoid an extra call to the database to check if the `parent_id` exists by defining the `path` directly in the `INSERT` and validating it afterwards.
…fields from parent In `creditOrDebitTabCoins`, changing the order of `if`s prevents a database query when the content is not relevant. Furthermore, searching the parent content directly with SQL allows to obtain only the desired field (`owner_id`), instead of bringing up all the fields from the table and also calculating the content's TabCoins.
Rafatcb
added
back
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
aprendendofelipe
previously approved these changes
Jun 21, 2024
There was a problem hiding this comment.
Boa @Rafatcb! 💪
Fiz alguns comentários, e subi commits com algumas das sugestões (não todas), mas pode remover o que você achar que não faz sentido.
E ao remover a updateMetadata na criação de conteúdo, eu percebi que dava para eliminar totalmente essa função do sistema, então já removi também do patch de usuários. Se achar que isso faz sentido, talvez compense editar o nome do PR, ou criamos outro, se achar melhor separar.
Rafatcb
changed the title
|
Ótimas melhorias, não percebi que poderíamos passar o |
Rafatcb
force-pushed
the
refactor/database-communication-contents
branch
from
11e7d84
to
583e89a
Compare
aprendendofelipe
approved these changes
Jun 25, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Mudanças realizadas
Realizei algumas alterações como tentativa de melhoria de desempenho na criação e edição de um conteúdo.
parentna query de criação ao invés de buscar separadamente.creditOrDebitTabCoins, mudar a ordem dosifs evita uma consulta no banco de dados quando o conteúdo não é relevante. Além disso, buscar o conteúdo pai diretamente com SQL permite obter apenas o campo desejado (owner_id), ao invés de trazer todos os campos da tabela e ainda calcular os TabCoins do conteúdo.creditOrDebitTabCoins, buscar o pai no banco de dados apenas casonewContent.parent_owner_idsejaundefinedpermite evitar buscar o pai duas vezes nocreate.PATCH, é possível definirmetadata.idao criar oevent, removendo a necessidade do uso doevent.updateMetadata. Além disso, ao aceitar receber umoptions.oldContentnocontent.update, evitamos buscar duas vezes o mesmo conteúdo durante oPATCH.Talvez alguma mudança seja desnecessária, possa ter uma interface melhor, ou ainda exista alguma melhoria que não percebi. Estou aberto a sugestões.
Tipo de mudança
Checklist: