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.

Sign up for GitHub

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

Jump to bottom

conn: OpenShardConn makes unnecessary retries #281

Open
Kulezi opened this issue Sep 7, 2022 · 0 comments · May be fixed by #295
Open

conn: OpenShardConn makes unnecessary retries #281

Kulezi opened this issue Sep 7, 2022 · 0 comments · May be fixed by #295
Labels
bug Something isn't working transport

Comments

@Kulezi
Copy link
Contributor

Kulezi commented Sep 7, 2022 •
edited
Loading

OpenShardConn should retry dial on other local port if the one it chose is busy.
However what happens is that it retries even when it got the port but the destination was unreachable / timeouted etc. bloating the logs unnecessarily.
@Kulezi Kulezi added bug Something isn't working transport and removed bug Something isn't working labels Sep 7, 2022
@Kulezi Kulezi mentioned this issue Sep 12, 2022
Closed
@Kulezi Kulezi added the bug Something isn't working label Sep 23, 2022
Kulezi added a commit that referenced this issue Sep 23, 2022
@Kulezi
transport: OpenShardConn, retry only when source port is unavailable …
31b0e68 
…for use

Fixes #281
@Kulezi Kulezi linked a pull request Sep 23, 2022 that will close this issue
Open
Kulezi added a commit that referenced this issue Sep 23, 2022
@Kulezi
transport: OpenShardConn, retry only when source port is unavailable …
8c54361 
…for use

Fixes #281
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working transport
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

transport: connection opening bugfixes scylladb/scylla-go-driver
1 participant
@Kulezi