samsondav / rihanna

Rihanna is a high performance postgres-backed job queue for Elixir
MIT License
439 stars 47 forks source link

Should due_at be part of the order in the lock query? #86

Closed tpitale closed 4 years ago

tpitale commented 5 years ago

Such as here: https://github.com/samphilipd/rihanna/blob/master/lib/rihanna/job.ex#L398

When a job is retried and set to due_at, should that value be considered at all? Should that be considered over the enqueued_at?

samsondav commented 5 years ago

@tpitale probably it should take higher priority than enqueued at. Makes sense to me.

samsondav commented 5 years ago

@tpitale PR incoming? ;)