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

Refactor: Move InstancePopulationAggregate calculations into the API #566

Open
Maelstromeous opened this issue Jul 12, 2022 · 0 comments
Open
Assignees
Labels
enhancement New feature or request

Comments

@Maelstromeous
Copy link
Member

So currently the InstancePopulationAggregate forces the aggregator to be exclusive per census environment. This logic should be refactored out of the Aggregator and moved into the API.

The API can then scan the redis keys like it does in Aggregator and send the data to the database directly. Plus we have much better control over the timings of the data in the API project as well, it's kinda shoehorned into the Aggregator atm.

This also enables us to have multiple aggregators per environment, whereas right now if one crashes, no data can be processed until it's restarted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant