sidekiq icon indicating copy to clipboard operation
sidekiq copied to clipboard
verified publisher icon sidekiq

show iterable state

Open joevandyk opened this issue 4 months ago • 3 comments

https://github.com/sidekiq/sidekiq/issues/6537

image DEVELOPMENT Sidekiq 2025-08-12 02-40-33 DEVELOPMENT Sidekiq 2025-08-12 02-41-38

joevandyk avatar Aug 12 '25 09:08 joevandyk

I like parts of this but the fundamental issue is that this adds a significant amount of network chatter to the Busy and Queue pages. If we are looking at a page of queued jobs which shows 50 jobs, currently that's 1 call to Redis to call LRANGE in paginator.rb. With this change, we're going to make 51 round trips. I think it is worth refactoring to take advantage of pipelining.

Busy is even worse because large users might have hundreds of jobs running concurrently.

mperham avatar Oct 22 '25 18:10 mperham

but the fundamental issue is that this adds a significant amount of network chatter to the Busy and Queue pages

I wonder if it is really needed on Busy and Queue pages? Imo, having it on a details page would be enough.

fatkodima avatar Nov 16 '25 09:11 fatkodima

I think it would be fine to add to the job details page.

mperham avatar Nov 16 '25 13:11 mperham