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

[Bug] Cannot use two different types of ProducerRegistry #3073

Closed
iancooper opened this issue May 6, 2024 · 0 comments
Closed

[Bug] Cannot use two different types of ProducerRegistry #3073

iancooper opened this issue May 6, 2024 · 0 comments
Assignees
Labels
3 - Done v9 Required for v9 release v10 Allocal to a v10 release

Comments

@iancooper
Copy link
Member

Describe the bug

If you want to use two different types of ProducerRegistry, this is not possible. We create the ProducerRegistry from the Producers that you want to register with us, via a Factory. Because the Factory is tied to the Transport (Kafka, RMQ) etc. you can't have a ProducerRegistry that contains multiple transports

Suggested Fix

We should enable the merging of Transport specific Producer Registries into a single Producer Registry. To do this we should add a Merge extension method for a Producer Registry, that takes another Producer Registry and returns a merged Producer Registry. By using an extension method we should be able to add this to both V9 and V10

@iancooper iancooper added 0 - Backlog v10 Allocal to a v10 release v9 Required for v9 release labels May 6, 2024
@iancooper iancooper self-assigned this May 6, 2024
@iancooper iancooper changed the title [BUG] Cannot use two different types of ProducerRegistry [Bug] Cannot use two different types of ProducerRegistry May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Done v9 Required for v9 release v10 Allocal to a v10 release
Projects
None yet
Development

No branches or pull requests

1 participant