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

Add Suppressed Alarms Count to Active tab #57

Open
slominskir opened this issue Nov 13, 2024 · 0 comments
Open

Add Suppressed Alarms Count to Active tab #57

slominskir opened this issue Nov 13, 2024 · 0 comments

Comments

@slominskir
Copy link
Member

Requested during ops review

Currently the Active tab creates a Server Sent Events connection on the effective-alarms topic, but filtered by initallyActiveOnly. To accommodate this request we may need to remove that server filter and perform client-side filtering. This may impact page performance, especially initial page load. The number of active alarms should be low in general, whereas loading all alarms (currently around 25 - 30,000) is much more costly in network load, server load, and page load time.

Also worth mentioning some Active alarms are "Incited", which could also be displayed in a separate count (though that is already part of the Active count).

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

No branches or pull requests

1 participant