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

Job that doesn't call done() retries twice, but attempts is set to 4 (ttl = 2000) #669

Closed
magtutu opened this issue Jul 16, 2015 · 4 comments
Labels

Comments

@magtutu
Copy link

magtutu commented Jul 16, 2015

I would expect the following job to be attempted 4 times. In 0.9.3 it is only run twice. In 0.9.1 the job is tried four times. TTL is a super important feature.

var kue = require('kue');
var queue = kue.createQueue();

queue.process('testjob', function(job, done){
  console.log('running testjob');
});

queue.create('testjob', {})
  .attempts(4)
  .ttl(2000)
  .save();
@magtutu
Copy link
Author

magtutu commented Jul 16, 2015

Redis 2.8.17 (00000000/0) 64 bit

@magtutu magtutu changed the title Job that doesn't call done() retries twice, but attempts is set to 4 Job that doesn't call done() retries twice, but attempts is set to 4 (ttl = 2000) Jul 16, 2015
@behrad behrad added the Bug label Jul 16, 2015
@behrad
Copy link
Collaborator

behrad commented Jul 16, 2015

this is a serious bug, will provide a patch soon. thank you @magtutu

@magtutu
Copy link
Author

magtutu commented Jul 17, 2015

ttl seems to work as expected in 0.9.4. Gracias!

@behrad
Copy link
Collaborator

behrad commented Jul 17, 2015

👍

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