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

MonitorResource consumes too many resources #18245

Closed
wezell opened this issue Apr 1, 2020 · 2 comments
Closed

MonitorResource consumes too many resources #18245

wezell opened this issue Apr 1, 2020 · 2 comments

Comments

@wezell
Copy link
Contributor

wezell commented Apr 1, 2020

In a production environment, the MonitorResource should be as light weight as possible and consume too many resources.

We have a large production environment where the status checks performed by the MonitorResource are consuming up to 10% of the total app resources, specifically the DB check. The DB check runs an unbounded count on the contentlet table and that performs a full table scan in order to return the value. See image below from a production site that is using the /api/v1/system-status endpoint

Screen Shot 2020-04-01 at 5 52 39 PM

@wezell
Copy link
Contributor Author

wezell commented Apr 7, 2020

PR: #18247

@dsilvam dsilvam self-assigned this Apr 14, 2020
@dsilvam
Copy link
Contributor

dsilvam commented Apr 14, 2020

Passed Internal QA: Profiler not showing bottlenecks in the db check

Screen Shot 2020-04-14 at 5 22 43 PM

dsilvam pushed a commit that referenced this issue Apr 14, 2020
* #18245 make monitor query lighter

* #18245 make monitor query lighter
@dsilvam dsilvam removed their assignment Apr 14, 2020
@wezell wezell closed this as completed May 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants