rabbitmq / rabbitmq-stream-dotnet-client

RabbitMQ client for the stream protocol
https://rabbitmq.github.io/rabbitmq-stream-dotnet-client/stable/htmlsingle/index.html
Other
122 stars 41 forks source link

Improve Reconnection #343

Closed Gsantomaggio closed 10 months ago

Gsantomaggio commented 10 months ago

How to test

/etc/hosts
127.0.0.1 node0
127.0.0.1 node1
127.0.0.1 node2
codecov[bot] commented 10 months ago

Codecov Report

Attention: 81 lines in your changes are missing coverage. Please review.

Comparison is base (a7c9fd6) 92.83% compared to head (10a3f5c) 91.87%.

Files Patch % Lines
RabbitMQ.Stream.Client/Reliable/ReliableBase.cs 75.47% 21 Missing and 5 partials :warning:
...bitMQ.Stream.Client/Reliable/IReconnectStrategy.cs 39.28% 17 Missing :warning:
RabbitMQ.Stream.Client/Reliable/Producer.cs 52.00% 7 Missing and 5 partials :warning:
RabbitMQ.Stream.Client/ClientExceptions.cs 16.66% 2 Missing and 3 partials :warning:
RabbitMQ.Stream.Client/ConnectionsPool.cs 86.48% 3 Missing and 2 partials :warning:
RabbitMQ.Stream.Client/Reliable/Consumer.cs 37.50% 3 Missing and 2 partials :warning:
RabbitMQ.Stream.Client/StreamSystem.cs 80.00% 5 Missing :warning:
RabbitMQ.Stream.Client/IConsumer.cs 0.00% 3 Missing :warning:
RabbitMQ.Stream.Client/Client.cs 96.29% 0 Missing and 1 partial :warning:
RabbitMQ.Stream.Client/Connection.cs 88.88% 0 Missing and 1 partial :warning:
... and 1 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #343 +/- ## ========================================== - Coverage 92.83% 91.87% -0.97% ========================================== Files 116 116 Lines 11084 11065 -19 Branches 913 915 +2 ========================================== - Hits 10290 10166 -124 - Misses 595 694 +99 - Partials 199 205 +6 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Gsantomaggio commented 10 months ago

@lukebakken Thanks for the help! Lock removed