[4.x] Do not record when unserializing command

[driesvints]

This PR prevents Telescope from recording duplicate queries when unserializing a job. Ideally the duplicate queries shouldn't happen but at least this prevents duplicate date from being added to Telescope.

Fixes #1255

[mpskovvang]

Thanks!

Perhaps it should check the original state before re-enabling recording?

[mpskovvang]

Some workaround ideas to completely avoid duplicate queries:

1. Use preg_match to extract the batch ID
2. Modify the JobProcessed event in laravel/framework to pass the unserialized data/payload (the job handler $instance property) using getResolvedJob method

Could we just do an instance check in the JobWatcher class with fallback to unserializing?

if ($event->job instanceof Job) {
    $event->job->getResolvedJob();
}

I can submit a PR if you like?

[driesvints]

Perhaps it should check the original state before re-enabling recording?

Thanks, I sent in a PR for that: #1258

Could we just do an instance check in the JobWatcher class with fallback to unserializing?

Sorry, I'm not sure if it's the right approach.