Skip to content

Introducing MultiRabbit Bootstrap #1303

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

Merged
merged 2 commits into from
Feb 8, 2021

Conversation

rwanderc
Copy link
Contributor

  • Creation of the MultiRabbitBootstrapConfiguration, activated via properties, since the default RabbitListenerBPP bean cannot be overridden with auto-config.
  • Injection of the MultiRabbit admin to the RabbitListener, before it's processed by RabbitListenerBPP, so as to have the bean resolved and assigned to the MethodRabbitListenerEndpoint. It fixes an issue lately found on the order of loading where the MultiRabbitAutoConfig (at boot) was not being processed before MultiRabbitBPP starts the postProcessAfterInitialization(), which leads to bean not found.
  • Move some configs to RabbitListenerConfigUtils as they are shared among Rabbit and MultiRabbit, as well as used in the boot.

Copy link
Contributor

@garyrussell garyrussell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty clean to me after a quick review. Just one comment.

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

Successfully merging this pull request may close these issues.

2 participants