search

lykmapipo / kue-scheduler

A job scheduler utility for kue, backed by redis and built for node.js
246 stars 47 forks source link

Adding 'ready' signal for handeling redis disconnects #63

Closed fruch closed 7 years ago

fruch commented 7 years ago

Don't merge this, it's not yet working as expected.

Too many different clients, doing this only on few of them isn't enough, we need this on all of them for it to actually work.

lykmapipo commented 7 years ago

@fruch Ok.

fruch commented 7 years ago

@lykmapipo I need help with this, seem like the event isn't enoguh to get up from redis disconnect. putting the jobs back (even if they are unique) isn't enough.

jobs aren't trigger again, after redis is connected.

fruch commented 7 years ago

@lykmapipo after trying it a lot of times, I've figured the redlock with retry=0 was the problem. and there's no need to use the signal from the redis clients.

I'm closing this one..