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

Move periodic tasks away from cron jobs #24

Open
Krishna-Sivakumar opened this issue Feb 27, 2022 · 4 comments
Open

Move periodic tasks away from cron jobs #24

Krishna-Sivakumar opened this issue Feb 27, 2022 · 4 comments
Labels
enhancement New feature or request
Milestone

Comments

@Krishna-Sivakumar
Copy link
Contributor

Krishna-Sivakumar commented Feb 27, 2022

Using celery sounds like a better option here.

https://docs.celeryproject.org/en/stable/

@kinshukdua
Copy link
Owner

kinshukdua commented Feb 27, 2022

PR #16 introduced the scheduler library as an alternative to cronjob for docker, although if celery is async, that would be a better solution.
Also I am not sure if using an external server like nginx or Apache is better vs using a small flask server and asynchronously updating the map with celery.

@Krishna-Sivakumar
Copy link
Contributor Author

Celery is asynchronous by nature

@Krishna-Sivakumar
Copy link
Contributor Author

Moving to a dynamic backend should be it's own issue

This was referenced Feb 27, 2022
@kinshukdua
Copy link
Owner

I think celery would be a better solution than scheduler library. It also saves CPU cycle on the backend, which would be better used for catering to more concurrent users.

@DomiiBunn DomiiBunn added this to the Beta 0.2.0 milestone Feb 28, 2022
@DomiiBunn DomiiBunn added the enhancement New feature or request label Feb 28, 2022
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

3 participants