-
Notifications
You must be signed in to change notification settings - Fork 36
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 up status page #1142
Speed up status page #1142
Conversation
…ent-parameter-in-status-page-to-speed-it-up
…ent-parameter-in-status-page-to-speed-it-up
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to combine the change with an assessment? From the issue:
Ideally, report back with the measured improvement on some production data dump.
I made a quick check without in-depth profiling (page with 8 sensors):
Promising. Still I believe this is a little slow, but this PR (well, the underlying work in timely-beliefs I did months ago) seems to make a notable improvement! |
What should be the next step? |
…-page-to-speed-it-up
I would say we merge this improvement and come back for more another time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Can we add a changelog entry under Bugfixes/Infrastructure?
Signed-off-by: Nicolas Höning <[email protected]>
Description
Used
most_recent_beliefs_only=False
andmost_recent_only=True
to speed up status page query.How to test