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

Correct outdated RabbitMQ listener properties #899

Merged

Conversation

Nkyn
Copy link
Contributor

@Nkyn Nkyn commented Oct 18, 2019

Since Spring AMQP version 2 the listener properties' names changed depending on the container that is used, e.g. spring.rabbitmq.listener.simple.key when using SimpleMessageListenerContainer.
For the prefetch property also the default value has changed to 250, which is set here explicitly to that value to stay stable in case of further changes.
The concurrency's min and max values are set from 1 (initial default) to 10 to avoid uncontrollably concurrency adjusting when the workload is increasing.

@hawkbit-bot
Copy link

Can one of the admins verify this patch?

@schabdo schabdo merged commit b50a82f into eclipse-hawkbit:master Oct 21, 2019
@schabdo schabdo deleted the feature_rabbitmq_listener_properties branch October 21, 2019 09:00
@schabdo schabdo added this to the 0.3.0M6 milestone Oct 21, 2019
@schabdo schabdo changed the title RabbitMQ listener properties Correct outdated RabbitMQ listener properties Jan 16, 2020
AmmarBikic pushed a commit to bosch-io/hawkbit that referenced this pull request Oct 2, 2020
* Rename dmf defaults properties to config rabbitmq's simple listener

Signed-off-by: Natalia Kislicyn <[email protected]>

* set rabbitmq listener prefetch and concurrency properties

Signed-off-by: Natalia Kislicyn <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants