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

Syncing will take more time as number of past jobs grows #52

Open
stevejalim opened this issue Jan 6, 2025 · 0 comments
Open

Syncing will take more time as number of past jobs grows #52

stevejalim opened this issue Jan 6, 2025 · 0 comments

Comments

@stevejalim
Copy link
Collaborator

In production, we don't delete completed jobs. As such, the amount of time and number of API calls made to sync against Smartling will only go up. At some point, without warning, this is likely to bite.

We should get ahead of this.

Ideas:

  • Don't re-check status of Completed jobs, saving us an API call for each -- this will only work if we never reopen completed jobs
  • See if we can filter the API call made to smartling to exclude Completed-status jobs -- but this would stop us changing state from In Progress to Completed, so it's more a case of excluding jobs that are Completed and older than a safety window of X hours (where X is more than a weekend, to be on the safe side)
  • More to come - suggestions welcome
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