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

Speed of Open Collective #6036

Open
4 tasks
shannondwray opened this issue Oct 7, 2022 · 7 comments
Open
4 tasks

Speed of Open Collective #6036

shannondwray opened this issue Oct 7, 2022 · 7 comments
Assignees
Labels
performance Load times, slow queries, overall execution speed
Milestone

Comments

@shannondwray
Copy link

Who is your user?

Everyone

What are they trying to achieve?

I've noticed lately that it is taking a while for the webpage to load and I'm frequently getting non responsive pages which I have to reload

How are they currently doing this?

Not to sure how this is currently monitored by the engineering team

How well understood is this problem?

  • Surfacing: if you feel this is a seed of an idea that needs to be researched.
  • Understanding: if you've done some research and feel it is already well understood and ready for shaping.
  • Shaping: if you feel a solution is clearly defined and can be made into a pitch (read more about shaping)
  • Pitching: if you feel that it is ready to be presented for prioritisation (read more about pitching)
@Betree Betree added the performance Load times, slow queries, overall execution speed label Oct 13, 2022
@Betree
Copy link
Member

Betree commented Oct 13, 2022

I share the feeling that, lately, multiple pages were really slow and sometimes unresponsive; usually on large hosts or OC Inc profiles. To come up with effective solutions, we need to better identify and document the pages that are slow/unresponsive.

In opencollective/opencollective-api#8051 we've plugged Sentry's performance tool; it should help us both to detect slow queries and to diagnose why they're slow.

We can use this issue to summarize our actions on this topic.

@Betree
Copy link
Member

Betree commented Jan 13, 2023

Search is extremely slow lately, would be great to investigate that area (related: #5809)

@gustavlrsn
Copy link
Member

One avenue to explore could be to statically render a lot of content, and fetch the logged in user data on the client.

Currently our approach is server side rendering everything, I think. But since most of the content on our platform is public, it seems like a good candidate to use static rendering.

We could use Next.js getStaticProps an on-demand revaidation to instantly revalidate a page when something gets outdated.

@kewitz
Copy link
Contributor

kewitz commented Mar 20, 2023

Project Updates Week 4:

  • Frontend performance metrics update merged and deployed;
  • Performance day at Berlin retreat:
    • Collective Page logic refactor specifications written and started;
    • Queries and resolvers reviewed;

Next steps:

  • Work on Collective Page refactor
  • Dig more into creating a follower DB outside Heroku

@znarf
Copy link
Member

znarf commented Mar 28, 2023

Last week.

Collective Page

Host Dashboard

CSV

@BenJam
Copy link
Contributor

BenJam commented Apr 3, 2023

closing as we've comitted a cycle (6 weeks) to addressing a number of concerns and increasing performance. We'll continue to look at performance and speed on a cycle by cycle basis but there's little value in keeping this open ad infinitum. Thank you and good night!

@shannondwray
Copy link
Author

@shannondwray shannondwray reopened this Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Load times, slow queries, overall execution speed
Projects
Status: ✅ Done
Development

No branches or pull requests

7 participants
@znarf @BenJam @gustavlrsn @Betree @kewitz @shannondwray and others