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

Process a single alert across all available threads #166

Open
mcoles opened this issue Aug 17, 2020 · 0 comments
Open

Process a single alert across all available threads #166

mcoles opened this issue Aug 17, 2020 · 0 comments

Comments

@mcoles
Copy link
Contributor

mcoles commented Aug 17, 2020

Tried to run a test_alert this morning on our server, and it worked the first time, but not the subsequent second two times. The problem was that a single alert was processing 50 different emails, all pulling separate versions of a viz down and rendering it to PNG. The viz in question took 25+ seconds to render, so for the duration of that time, VizAlerts could not pick up new alerts to process. We should change VizAlerts from being single-threaded within a single alert, and make it use all available threads for any and all alerts. Another thing we can do, which should probably be in its own Issue, is alert VizAlerts so that it runs as a service and can scan for new alerts that come in, while it processes the existing ones.

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

1 participant