go-workers icon indicating copy to clipboard operation
go-workers copied to clipboard

queue is not FIFO, but LIFO actually..

Open aki-s opened this issue 3 years ago • 1 comments

Version of go-workers

v0.0.0-20180112190529-dbf81d0b75bb

Details

  • line.86 in workers.fetcher.go
  • line.81 in workers.enqueue.go

image

My concern

I don't know the spec of Sidekiq, but I suspect we should use lpush instead of rpush to be FIFO. What do you think Mr. @jrallison ?

It looks Sidekiq is actually FIFO https://github.com/mperham/sidekiq/issues/3420 .

aki-s avatar Apr 07 '21 07:04 aki-s

I heard from my co-worker that fork project of this one has fixed the issue I've mentioned at https://github.com/digitalocean/go-workers2/pull/5 .

aki-s avatar Apr 08 '21 02:04 aki-s