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

Melhora na performance da paginação de conteúdos #1141

Closed
filipedeschamps opened this issue Dec 19, 2022 · 2 comments · Fixed by #1822
Closed

Melhora na performance da paginação de conteúdos #1141

filipedeschamps opened this issue Dec 19, 2022 · 2 comments · Fixed by #1822
Labels
back Envolve modificações no backend

Comments

@filipedeschamps
Copy link
Owner

Contexto

A paginação de conteúdos do TabNews utiliza a estratégia de offset, onde é uma estratégia simples de se implementar, porém possui um tradeoff de performance que é precisar percorrer por todos os itens do offset, mesmo que o limite de itens a ser devolvido seja 1. E eu especulo que isso se agrava, pois para cada linha é computado o get_current_balance(), o que para algumas páginas, não dava mais tempo de processar o retorno dentro de 60 segundos, retornando para o client um 504.

Execução

@gpoleszuk
Copy link

Prezado desenvolvedor.

Primeiro, peço desculpas pela falta de acentuaç~ao.

Reporte:
Ao tentar atualizar um rascunho em um comentario, o servidor retornou HTTP 504 apos 60 segundos de espera. Repeti a tentativa algumas vezes pensando ser problema por aqui no navegador. Entao, para gerar evidencias para voces, repeti o processo novamente em 20241129T084200Z utilizando o curl com a requisiçao gerada pelo navegador. Eu acho que ha algo no conteudo, pois postar algo mais simples, foi rapidamente aceito pelo servidor. A postagem ja foi removida.

Caso seja de vosso interesse, posso enviar a requisiçao POST completa que foi enviada para o servidor. Por hora, deixo uma versao sem os cookies de secao nesta postagem temporaria https://pastebin.com/dbFGUd0w (password: bmKQBJPiqH). Expira em pouco menos de 24 horas.

@aprendendofelipe
Copy link
Collaborator

@gpoleszuk, muito obrigado por reportar! 💪

O problema foi resolvido com o PR #1822 🤝

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back Envolve modificações no backend
Projects
None yet
3 participants