Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Support for binding multiple queues to the same exchanges #227

Merged

Conversation

BatyaPinski
Copy link

@BatyaPinski BatyaPinski commented May 24, 2023

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)
  • WHOSUSING.md
  • Other (please describe):

Changes in this PR

Support for binding multiple queues to the same exchange -
Currently, Conductor support events handler to be bound to an exchange , but the problem is that the current code does not get the queue name to be bound to the exchange and it creates the queue with default name.
That means that event handlers with the same exchnage will be bound to the same queue.
In this PR, we added 'bindQueueName' param to get the queue name from the exchange uri to enable multiple event handlers to be bound to the same exchange with different queues.

@v1r3n v1r3n merged commit c0fe98c into Netflix:main Jun 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants