Publish multiple messages - Batch publish

[filipbekic01]

I'm having trouble pushing 200+ messages at once. Each publish takes ~2 seconds, that's really slow.

[joskfg]

I'm having the same issue and this library doesn't support it. It uses basic_publish and there isn't any option to use batch_basic_publish.

[filipbekic01]

I'd love to extend possibilities but i'm not even sure how active this repository is. I see there are 3 PRs in queue already :-1:

[bschmitt]

The library doesn't support batch_basic_publish for now. But feel free to provide a PR. There are 3 PRs in review, some of them don't follow the intention of this library.

[stevenklar]

Hey Filip, I would love to merge any appropriate enhancement to the repository. Just create a PR and follow the repositories purpose.

Thank you :-)

On Wed, 7 Oct 2020 at 21:12, Björn Schmitt notifications@github.com wrote:

The library doesn't support batch_basic_publish for now. But feel free to provide a PR. There are 3 PRs in review, some of them don't follow the intention of this library.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/bschmitt/laravel-amqp/issues/91#issuecomment-705138824, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAC4WAWXG6R5VHC2ZXOQRL3SJS4TLANCNFSM4SE6N2JQ .

[joskfg]

I created a version of this library, with the support. After further testing, I'd like to put here as PR.

[filipbekic01]

Do that @joskfg :+1:

[joskfg]

Initial approach https://github.com/bschmitt/laravel-amqp/pull/93

[stevenklar]

Thank you for the contribution. I would like to merge this for our next release. Please fix the necessary docblocks and appropriate method scope.

-Steven

[joskfg]

Fixed docblocks and method scope.

[stevenklar]

Will be published in the next release.