Concurrency: Allow custom timeout settings

[clementbirkle]

Currently, it's not possible to run tasks in Concurrency that take longer than 60 seconds to complete. This limitation stems from this line in the Laravel framework:
https://github.com/laravel/framework/blob/11.x/src/Illuminate/Process/PendingProcess.php#L44

It would be helpful to allow custom timeouts for tasks. For example:

$results = Concurrency::timeout(300)->run(...);

This way, developers could set a timeout for long-running tasks as needed.

[henzeb]

EDIT: looking into the code, I see public function timeout(int $timeout) isn't that what you were looking for?

EDIT2: it also has public function forever()

[clementbirkle]

Unfortunately, we can't interact with this class directly when using the Concurrency facade.

For example this code:

[$userCount, $orderCount] = Concurrency::timeout(300)->run([
    fn () => 10,
    fn () => 20,
]);

Throw this error:

Call to undefined method Illuminate\Concurrency\ProcessDriver::timeout()

[ahuggins]

Can confirm, it looks as if these methods are there, but also seems as if we can not use them. Wonder if it is possible to pass a config item or set a config before calling Concurrency::run()?

[sebj54]

A bit late to the party, but here is what I did:

app()->bind(Illuminate\Process\Factory::class, function () {
    return new class extends Illuminate\Process\Factory
    {
        public function newPendingProcess()
        {
            return parent::newPendingProcess()->forever();
        }
    };
});

You can use this snippet at the very beginning of a controller or globally in AppServiceProvider for instance.