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

lock remains after job not properly finish #339

Closed
mathieujobin opened this issue Nov 2, 2018 · 4 comments
Closed

lock remains after job not properly finish #339

mathieujobin opened this issue Nov 2, 2018 · 4 comments

Comments

@mathieujobin
Copy link

Describe the bug
We have had this issue with one specific job, where the unique lock would remain in redis after the job has crashed or did not finish properly

is this the intended behavior?

Expected behavior
the lock should be cleared after the job finish, regardless of the status

Current behavior
new jobs gets schedule and are discarded indefinitely

Worker class

class MyWorker
  include Sidekiq::Worker

  sidekiq_options :retry => false, :backtrace => true, :unique => :while_executing

  def perform(args); raise("I made a mistake"); end
end

Additional context
I noticed you are talking there was a 30 minutes timeout in the 5.x series which got removed.
but in your readme you seems to be recommending against using lock_expiration/lock_timeout
I am also unsure which one would fit best my use case.

thank you for your help

@mathieujobin
Copy link
Author

oh, this is exact duplicate of #332

@mathieujobin
Copy link
Author

reopening, I'd appreciate some insight as to know if lock_expiration or lock_timeout could help ?

@mathieujobin mathieujobin reopened this Nov 2, 2018
@mhenrixon
Copy link
Owner

Lock expiration definitely will help. The timeout is just how long we wait for a lock to be achieved

@mathieujobin
Copy link
Author

thanks

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

2 participants