pkg/grpc/client.go #43

New issue

Closed

opened 2026-03-28 04:20:43 +00:00 by mfreeman451 · 0 comments

mfreeman451 commented

2026-03-28 04:20:43 +00:00

Owner

Imported from GitHub.

Original GitHub issue: #80
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/issues/80
Original created: 2025-01-20T03:46:33Z

The maxRetries field in ClientConn is not used by the RetryInterceptor. The retry logic always uses the defaultMaxRetries.
The lastHealthDetails and lastHealthCheck fields could be protected by a separate mutex to avoid potential data races.
The error messages could be made more informative.

Imported from GitHub. Original GitHub issue: #80 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/issues/80 Original created: 2025-01-20T03:46:33Z --- - [ ] The maxRetries field in ClientConn is not used by the RetryInterceptor. The retry logic always uses the defaultMaxRetries. - [ ] The lastHealthDetails and lastHealthCheck fields could be protected by a separate mutex to avoid potential data races. - [ ] The error messages could be made more informative.

Rows
Columns