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 issue on admin dashboard #15953

Open
vasilvestre opened this issue Mar 6, 2024 · 5 comments
Open

Performance issue on admin dashboard #15953

vasilvestre opened this issue Mar 6, 2024 · 5 comments
Assignees
Labels
Admin AdminBundle related issues and PRs. Performance

Comments

@vasilvestre
Copy link
Contributor

vasilvestre commented Mar 6, 2024

Describe the proposed solution
When you have a lot of data, the dashboard can take up to 5-10 sec more depending of data. It slow the login process when you do not even need to get stats.
What could be done :

  • toggle to enable/disable stats (not everyone use it)
  • cache data for X minutes/hours
  • move stats to another tab

Additional context
Here's a Sentry analysis of the time :
Capture d’écran du 2024-03-06 10-50-03

You can see the statistics do not contain code overwrite

@TheMilek
Copy link
Member

TheMilek commented Mar 8, 2024

Hi @vasilvestre!

Thank you for opening the issue,
Could you provide more information about the amount of data you are relying on?

Thanks in advance! 🍻

@TheMilek TheMilek added Performance Admin AdminBundle related issues and PRs. labels Mar 8, 2024
@vasilvestre
Copy link
Contributor Author

Hi @vasilvestre!

Thank you for opening the issue,
Could you provide more information about the amount of data you are relying on?

Thanks in advance! 🍻

More than 10000 for this one year so 3-5k per month

@Rafikooo
Copy link
Contributor

Hello @vasilvestre. We would like to inform you that we have discussed your concern and acknowledge its validity, however, it is not currently among our top priorities. It is possible that we will revisit this issue around the time of the Sylius 2.0 release 💃

@vasilvestre
Copy link
Contributor Author

Hello @vasilvestre. We would like to inform you that we have discussed your concern and acknowledge its validity, however, it is not currently among our top priorities. It is possible that we will revisit this issue around the time of the Sylius 2.0 release 💃

I've found a solution, queries and indexes seem suboptimal. I will share the changes !

@TakesTheBiscuit
Copy link

@vasilvestre we would be interested in this also

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin AdminBundle related issues and PRs. Performance
Projects
None yet
Development

No branches or pull requests

4 participants