search

fudgebucket27 / Lexplorer

Loopring explorer alternative
14 stars 10 forks source link

Speed up transactions query #177

Closed modersohn closed 2 years ago

modersohn commented 2 years ago

Getting transactions takes long, whether on home page, accounts page or transactions itself.

I suspect the query is way to big, due to the nesting of the various fragments. This should be optimised, so only data that is actually displayed in the overview tables is queried.

modersohn commented 2 years ago

Looking good so far, reduced response size from 2kB per transaction to about half!

daniel-soli commented 2 years ago

That's great! Halvening of response size and query would help much!

fudgebucket27 commented 2 years ago

Wow that's amazing!

On Sun, 26 June 2022, 3:16 am Daniel Soli Stensvik, < @.***> wrote:

That's great! Halvening of response size and query would help much!

— Reply to this email directly, view it on GitHub https://github.com/fudgebucket27/Lexplorer/issues/177#issuecomment-1166328250, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABIDWTZOV4Q4LGTXDR5Q5X3VQ45H5ANCNFSM5ZL65VBA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

modersohn commented 2 years ago

This is live now on dev deployment for a while so I'll close this. I've seen very fast response times when browsing through transactions, but that could very well also be caused by the graph being more performant at times.