Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consumer 'duration' option #347

Open
guyradford opened this issue Nov 22, 2017 · 1 comment
Open

Consumer 'duration' option #347

guyradford opened this issue Nov 22, 2017 · 1 comment

Comments

@guyradford
Copy link

guyradford commented Nov 22, 2017

I see that the dequeue duration is hardcoded to 5 seconds. This time cant be overridden anywhere. With such a short long poll wait we get lots of logs and we are only running 5 different job workers.

Would it make sense to extend the Consumer class allowing another options value to be added.

        // Process queue of messages
        $this->getConsumer($app)->consume($queue, [
            'stop-when-empty' => false,
            'duration' => 30

        ]);

The Consumer class can then be updated to accept this new option as pass it to the $queue->dequeue() method.

Thank you
Guy

@rtripault
Copy link

Just to note that's somehow duplicate of #319

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants