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.

Sign up for GitHub

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

Jump to bottom

Jobs sometimes executed multiple times. #13

Closed
sterien7 opened this issue Oct 23, 2024 · 0 comments · Fixed by #14
Closed

Jobs sometimes executed multiple times. #13

sterien7 opened this issue Oct 23, 2024 · 0 comments · Fixed by #14

Comments

@sterien7
Copy link

Hi team,

I started using the driver and I noticed that there are some Jobs running twice.
I see an open issue in the forked repo since 2022

m-barthelemy#27
gwynne added a commit that referenced this issue Oct 24, 2024
@gwynne
Add an "initial" state to jobs to avoid race conditions between the s…
d2388b1 
…et() and push() queue operations. Fixes #13
@gwynne gwynne mentioned this issue Oct 24, 2024
Merged
gwynne added a commit that referenced this issue Oct 24, 2024
@gwynne
Fix race condition between set() and push() (#14)
20ebbf1 
A worker can end up picking up a job between `set()` and `push()`,
causing a race condition that results in the job running twice. Fix
the race by adding an `initial` state for jobs that haven't yet
been pushed.

Fixes #13.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix race condition between set() and push() vapor-community/vapor-queues-fluent-driver
1 participant
@sterien7