michaelklishin / quartz-mongodb

A MongoDB-based store for the Quartz scheduler. This fork strives to be as feature complete as possible. Originally by MuleSoft.
Other
249 stars 203 forks source link

Small fix related to previous change #9

Closed vladvoic closed 11 years ago

vladvoic commented 11 years ago

My previous changed assumed that the expired locks are only tried to be removed by the same instance that created them. This is not true, so I just changed that locks are removed only based on their id and not based on the instance ID.

I think this is desired, as the instance id could easily change from one run to another, and the expired locks would be zombies as they are not matched by the new instance id.

Please feel free to suggest another approach to this if you feel this is not right.