Skip to content

[13.x] Add queue to InspectedJob#60374

Merged
taylorotwell merged 5 commits into
laravel:13.xfrom
jackbayliss:13.x-add-queue-to-inspected-job
Jun 4, 2026
Merged

[13.x] Add queue to InspectedJob#60374
taylorotwell merged 5 commits into
laravel:13.xfrom
jackbayliss:13.x-add-queue-to-inspected-job

Conversation

@jackbayliss
Copy link
Copy Markdown
Contributor

@jackbayliss jackbayliss commented Jun 3, 2026

When you do Queue::allPendingJobs() or any of the all methods, it would be mighty useful to see the queue the job is on. This is very useful in general for debugging or clarifying behavior in live data.

The database one was straight forward

The redis one was a bit of a pain and has few changes, but I de-duplicated some bits, essentially just makes the queue name available.

I've put queue prop b4 created_at, which should be fine as all this is fairly new.

Sorry I keep tweaking this, I keep reaching for it for different bits and realize I wanna see more. This is the last of it 🗡️ (at least for awhile 🙇🏻 )

Feel free to tweak as perhaps you can see somethin I cant!

Update InspectedJob.php

database

fake

fake test

this is the same

redis queue name bits

specific cluster test
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

Thanks for submitting a PR!

Note that draft PRs are not reviewed. If you would like a review, please mark your pull request as ready for review in the GitHub user interface.

Pull requests that are abandoned in draft may be closed due to inactivity.

I imagine I will forget this in 3 weeks
@jackbayliss jackbayliss marked this pull request as ready for review June 4, 2026 09:27
@taylorotwell
Copy link
Copy Markdown
Member

I feel like this should come right after $uuid in terms of argument order.

@taylorotwell taylorotwell merged commit bfbf5cf into laravel:13.x Jun 4, 2026
53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants