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

[∞] Performance issues #1533

Open
nmorduch opened this issue Jun 10, 2020 · 5 comments · Fixed by #1536
Open

[∞] Performance issues #1533

nmorduch opened this issue Jun 10, 2020 · 5 comments · Fixed by #1536
Labels

Comments

@nmorduch
Copy link
Contributor

  • Slow loading times
    • weekly page took 36 seconds for everything below the main toolbar to load whereas the main Publications page only took 5 seconds
    • The content grid on the Event page took 17 seconds
  • Memory overload for web dynos
    • I just decreased to 5 workers but not sure this will help
  • System exits in Sentry might be related?
@kaedroho
Copy link
Contributor

kaedroho commented Jun 11, 2020

Naomi and I had a look through New Relic during our catchup today:

  • The Weekly takes an average of 7 seconds to load. A lot of time is spent in Python and template rendering. It runs about 150 queries for images so I think it's most likely to be image processing causing the issue. We don't think this view should be fetching any images though because the listing is rendered with React so this might be an easy fix
  • The Meta API is called on every single request and takes over a second each time to respond. If this is returning the same response for every user we can cache this.
  • The system exits are because the Gunicorn workers are timing out. I think there's a chance this might be reduced/eliminated when we fix the meta API (which currently uses up around 50% of total request time on its own).

@kaedroho
Copy link
Contributor

kaedroho commented Jun 25, 2020

We had another look at this. There's also a performance issue with editing reports. Sometimes the page editor times out and the page can't be edited.

According to New Relic, It's spending a lot of time rendering streamfield blocks. Switching to wagtail-react-streamfield might solve this.

@nmorduch nmorduch linked a pull request Aug 19, 2020 that will close this issue
@nmorduch
Copy link
Contributor Author

Search still has problems, probably fixed in #1494. Post api is still having problems.

@nmorduch nmorduch added the 2 med label Jan 19, 2021
@nmorduch
Copy link
Contributor Author

Changed to medium priority because we've made some progress on this but there's still more we could do. ie it's still bad but not abominable.

@nmorduch nmorduch changed the title Performance issues [∞] Performance issues Jan 19, 2021
@nmorduch nmorduch removed their assignment Feb 16, 2021
@nmorduch nmorduch added someday and removed 2 med labels Apr 16, 2021
@nmorduch
Copy link
Contributor Author

This is not worth it in the near future if we're redoing our FE

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

Successfully merging a pull request may close this issue.

3 participants