Skip to content

RabbitMQ: shared connections with context safety and config mismatch guard#3353

Closed
ozangunalp wants to merge 4 commits intosmallrye:mainfrom
ozangunalp:CassioGangrel/main
Closed

RabbitMQ: shared connections with context safety and config mismatch guard#3353
ozangunalp wants to merge 4 commits intosmallrye:mainfrom
ozangunalp:CassioGangrel/main

Conversation

@ozangunalp
Copy link
Collaborator

Clone of #3266

@ozangunalp
Copy link
Collaborator Author

@CassioGangrel, sorry the PR was closed, I don't know what happened there.

I pushed some tests and quick fixes to the branch.
I was fearing that the connection creation had some race conditions.
I reviewed the ClientHolder globally. There was a lot of cleanup to do.

@cescoffier, if you'd have time to take a look, I am ok with this to be merged if the CI is green.

@ozangunalp ozangunalp requested a review from cescoffier March 27, 2026 09:50
@ozangunalp
Copy link
Collaborator Author

CI green. Will follow up with another PR to improve RabbitMQ test coverage.

@ozangunalp ozangunalp added this to the 4.34.0 milestone Mar 27, 2026
@ozangunalp
Copy link
Collaborator Author

Closed with #3354

@ozangunalp ozangunalp closed this Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant