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

Adjust cronjob/batch update logic to match new DB size #74

Open
foolo opened this issue Apr 9, 2024 · 2 comments
Open

Adjust cronjob/batch update logic to match new DB size #74

foolo opened this issue Apr 9, 2024 · 2 comments
Labels

Comments

@foolo
Copy link
Contributor

foolo commented Apr 9, 2024

Need to investigate how the cronjob should work with a 1M records database after #66

A cronjob may not be the ideal solution. An alternative is to use a constantly running process.

@foolo foolo changed the title Adjust cron job numbers to match new DB size Adjust cronjob/batch update logic to match new DB size Apr 9, 2024
@Divide-By-0
Copy link
Member

I noticed that the current job has been shut off, but it might be good to keep that on for just emails that were found in inbox so far, until we resolve this issue, as those are the most important ones.

@foolo
Copy link
Contributor Author

foolo commented Apr 23, 2024

I noticed that the current job has been shut off, but it might be good to keep that on for just emails that were found in inbox so far, until we resolve this issue, as those are the most important ones.

Okay! we can update the cronjob so that it only checks for records which are not marked as "scraped", etc

It is running though (i double checked in crontab, and in server logs and it's all up and running), but only with 10 DSPs every 1 minute, so with the current database size of 1M, it's basically negligible :) But as you suggested, we can change to only "from inbox" records, and then it will make sense again, when the number of records in scope will be below 10k or so.

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

No branches or pull requests

2 participants