-
Notifications
You must be signed in to change notification settings - Fork 6
RabbitMQ Cluster: make quorum default queue type ⚠️
#1238
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
RabbitMQ Cluster: make quorum default queue type ⚠️
#1238
Conversation
Make sure that durable queues are quorum instead of classic (see rabbitmq/rabbitmq-server#14763). WARNING: this does not enforce queue type. Certain queue parameter may lead to a different type (e.g. exclusive queue are always classic) Related issue/s * ITISFoundation#1232
GitHK
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK with this, beware about the driver somehow overwriting the option (but this needs checking)
…te-default-queue-type
mrnicegyu11
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Acknowledged , don't have a lot of experience here, lgtm
sanderegg
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
quorum default queue typequorum default queue type ⚠️
This reverts commit 797bd15.
What do these changes do?
Make sure that durable queues are quorum instead of classic (see rabbitmq/rabbitmq-server#14763).
WARNING: this does not enforce queue type. Certain queue parameter may lead to a different type (e.g. exclusive queue are always classic)
Related issue/s
Related PR/s
Manual actions
For changes to take affect, update virtualhost metadata via rabbitmqctl
Checklist