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

Developer logs can really slam frontend- might be better off by default. #3057

Closed
robinsowell opened this issue Mar 1, 2023 · 0 comments · Fixed by #3212
Closed

Developer logs can really slam frontend- might be better off by default. #3057

robinsowell opened this issue Mar 1, 2023 · 0 comments · Fixed by #3212

Comments

@robinsowell
Copy link
Contributor

For really big, active sites, the developer log can quickly get unwieldy and cause a slowdown on the frontend.

We run developer() on the frontend, which caches the entire contents of the dev log table, and that can get huge and it causes a noticable slowdown in some cases.

It seems to me it should be off by default, and something you turn on during developement or troubleshooting, not something that needs to be running routinely.

We're already making attempts to optimize it- the whole developer function works hard to not update if there's not already an existing, unexpired message. We could perhaps improve performance by automatically pruning more. But if you have a lot of dev reports coming in, it's just an exercise in dropping data and adding more data and caching thousands or rows in an effort to not continually insert more rows.

Really useful for debug, but just having it off by default seems like it makes more sense and speeds things up for a live site.

intoeetive added a commit that referenced this issue Apr 4, 2023
intoeetive added a commit that referenced this issue Apr 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant