codecov[bot]
commented
10 months ago Codecov Report
Attention: 10 lines in your changes are missing coverage. Please review.
Comparison is base (
083ff20) 91.96% compared to head (9130e38) 91.86%.
Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #345 +/- ## ========================================== - Coverage 91.96% 91.86% -0.11% ========================================== Files 116 116 Lines 11264 11320 +56 Branches 895 897 +2 ========================================== + Hits 10359 10399 +40 - Misses 701 718 +17 + Partials 204 203 -1 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Per conversation with @acogoluegnes we want to have (more or less) the same Stream Java Client behaviour.
The back-off reconnect strategy is increased sensibly. The previous value was too aggressive. It caused a lot of tentative and a lot of server requests. The TCP port can be ready during the restart, but the stream cannot be ready due to the sync.
Introduce a random delay on the strategy to avoid having the same reconnection time in case the client has more producers and consumers.
Introduce also a random delay on the disconnected part on disconnect for the same above reason.
Fixes a bug: check the leader during the reconnection partition.
Introduce the TestBackOff strategy for the tests. The default value is too high for the tests