Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(job-scheduler): validate if next delayed job exists #3015

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

roggervalf
Copy link
Collaborator

@roggervalf roggervalf commented Jan 16, 2025

Why

  1. What problem does it solve or improve? Jobs cannot be in more than 1 state, if we do not validate if a job already exists when a delayed job that is created by a job scheduler is promoted, we can have this job id in more than 1 state, i.e active and delayed or completed and delayed

How

Enter the implementation details here.

  1. How did you implement this? Validate job key existence before trying to add this record

Additional Notes (Optional)

Any extra info here.

@roggervalf roggervalf force-pushed the fix-scheduler-no-add-existing-job branch 2 times, most recently from de2a4b3 to f251f8b Compare January 25, 2025 18:20
@roggervalf roggervalf force-pushed the fix-scheduler-no-add-existing-job branch from 94f37ff to 5e9015a Compare January 26, 2025 02:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant