You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iggy.apache.org by "spetz (via GitHub)" <gi...@apache.org> on 2026/03/05 18:49:09 UTC
[PR] fix(rust): auto-reconnect on transport errors after server restart [iggy]
spetz opened a new pull request, #2880:
URL: https://github.com/apache/iggy/pull/2880
The SDK's send_raw_with_response() only triggered reconnect for a
subset of errors (Disconnected, EmptyResponse, Unauthenticated,
StaleClient). Transport-specific write errors like TcpError,
ConnectionClosed, WebSocketSendError, and QuicError were not in
the list, causing producers and consumers to loop forever on a
dead connection instead of reconnecting.
Added all transport-specific errors to the reconnect match in
TCP, WebSocket, and QUIC clients. TLS connections use the same
error types and code path, so they are also covered.
Integration tests verify both producer and consumer reconnect
across all three stateful transports (TCP, WebSocket, QUIC).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(rust): auto-reconnect on transport errors after server restart [iggy]
Posted by "spetz (via GitHub)" <gi...@apache.org>.
spetz merged PR #2880:
URL: https://github.com/apache/iggy/pull/2880
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(rust): auto-reconnect on transport errors after server restart [iggy]
Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #2880:
URL: https://github.com/apache/iggy/pull/2880#issuecomment-4007015962
## [Codecov](https://app.codecov.io/gh/apache/iggy/pull/2880?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 66.31%. Comparing base ([`6735eea`](https://app.codecov.io/gh/apache/iggy/commit/6735eea6bc8e30c889981e45ac4c881f56396d4e?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`861a948`](https://app.codecov.io/gh/apache/iggy/commit/861a948125ef92eac23460d1fafc5181be7cbab0?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #2880 +/- ##
============================================
- Coverage 68.33% 66.31% -2.03%
Complexity 739 739
============================================
Files 1052 977 -75
Lines 84745 78887 -5858
Branches 61281 61105 -176
============================================
- Hits 57910 52310 -5600
+ Misses 24463 24205 -258
Partials 2372 2372
```
| [Flag](https://app.codecov.io/gh/apache/iggy/pull/2880/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [node](https://app.codecov.io/gh/apache/iggy/pull/2880/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `87.29% <ø> (-4.99%)` | :arrow_down: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Files with missing lines](https://app.codecov.io/gh/apache/iggy/pull/2880?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [core/sdk/src/quic/quic\_client.rs](https://app.codecov.io/gh/apache/iggy/pull/2880?src=pr&el=tree&filepath=core%2Fsdk%2Fsrc%2Fquic%2Fquic_client.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-Y29yZS9zZGsvc3JjL3F1aWMvcXVpY19jbGllbnQucnM=) | `71.20% <ø> (ø)` | |
| [core/sdk/src/tcp/tcp\_client.rs](https://app.codecov.io/gh/apache/iggy/pull/2880?src=pr&el=tree&filepath=core%2Fsdk%2Fsrc%2Ftcp%2Ftcp_client.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-Y29yZS9zZGsvc3JjL3RjcC90Y3BfY2xpZW50LnJz) | `72.66% <ø> (ø)` | |
| [core/sdk/src/websocket/websocket\_client.rs](https://app.codecov.io/gh/apache/iggy/pull/2880?src=pr&el=tree&filepath=core%2Fsdk%2Fsrc%2Fwebsocket%2Fwebsocket_client.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-Y29yZS9zZGsvc3JjL3dlYnNvY2tldC93ZWJzb2NrZXRfY2xpZW50LnJz) | `61.00% <ø> (ø)` | |
... and [93 files with indirect coverage changes](https://app.codecov.io/gh/apache/iggy/pull/2880/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
</details>
<details><summary> :rocket: New features to boost your workflow: </summary>
- :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
- :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org