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

Change discoverer client to broadcast to read replicas #2276

Merged

ykadowak merged 14 commits into main from feature/readreplica/client

Jan 19, 2024

Merged

Change discoverer client to broadcast to read replicas #2276

Merge branch 'main' into feature/readreplica/client

Codecov / codecov/patch succeeded Jan 19, 2024 in 1s

44.31% of diff hit (target 30.11%)

View this Pull Request on Codecov

44.31% of diff hit (target 30.11%)

Annotations

Check warning on line 96 in internal/client/v1/client/discoverer/discover.go

codecov / codecov/patch

internal/client/v1/client/discoverer/discover.go#L91-L96

Added lines #L91 - L96 were not covered by tests

Check warning on line 159 in internal/client/v1/client/discoverer/discover.go

codecov / codecov/patch

internal/client/v1/client/discoverer/discover.go#L159

Added line #L159 was not covered by tests

Check warning on line 75 in internal/client/v1/client/discoverer/option.go

codecov / codecov/patch

internal/client/v1/client/discoverer/option.go#L71-L75

Added lines #L71 - L75 were not covered by tests

Check warning on line 157 in internal/client/v1/client/discoverer/option.go

codecov / codecov/patch

internal/client/v1/client/discoverer/option.go#L153-L157

Added lines #L153 - L157 were not covered by tests

Check warning on line 86 in internal/config/lb.go

codecov / codecov/patch

internal/config/lb.go#L74-L86

Added lines #L74 - L86 were not covered by tests

Check warning on line 72 in pkg/gateway/lb/handler/grpc/aggregation.go

codecov / codecov/patch

pkg/gateway/lb/handler/grpc/aggregation.go#L72

Added line #L72 was not covered by tests

Check warning on line 98 in pkg/gateway/lb/handler/grpc/handler.go

codecov / codecov/patch

pkg/gateway/lb/handler/grpc/handler.go#L98

Added line #L98 was not covered by tests

Check warning on line 1656 in pkg/gateway/lb/handler/grpc/handler.go

codecov / codecov/patch

pkg/gateway/lb/handler/grpc/handler.go#L1656

Added line #L1656 was not covered by tests

Check warning on line 2565 in pkg/gateway/lb/handler/grpc/handler.go

codecov / codecov/patch

pkg/gateway/lb/handler/grpc/handler.go#L2565

Added line #L2565 was not covered by tests

Check warning on line 2781 in pkg/gateway/lb/handler/grpc/handler.go

codecov / codecov/patch

pkg/gateway/lb/handler/grpc/handler.go#L2781

Added line #L2781 was not covered by tests

Check warning on line 2908 in pkg/gateway/lb/handler/grpc/handler.go

codecov / codecov/patch

pkg/gateway/lb/handler/grpc/handler.go#L2908

Added line #L2908 was not covered by tests

Check warning on line 3146 in pkg/gateway/lb/handler/grpc/handler.go

codecov / codecov/patch

pkg/gateway/lb/handler/grpc/handler.go#L3146

Added line #L3146 was not covered by tests

Check warning on line 85 in pkg/gateway/lb/service/gateway.go

codecov / codecov/patch

pkg/gateway/lb/service/gateway.go#L80-L85

Added lines #L80 - L85 were not covered by tests

Check warning on line 88 in pkg/gateway/lb/service/gateway.go

codecov / codecov/patch

pkg/gateway/lb/service/gateway.go#L88

Added line #L88 was not covered by tests

Check warning on line 67 in pkg/gateway/lb/usecase/vald.go

codecov / codecov/patch

pkg/gateway/lb/usecase/vald.go#L66-L67

Added lines #L66 - L67 were not covered by tests

Check warning on line 103 in pkg/gateway/lb/usecase/vald.go

codecov / codecov/patch

pkg/gateway/lb/usecase/vald.go#L103

Added line #L103 was not covered by tests

Check warning on line 244 in pkg/index/job/readreplica/rotate/service/rotator.go

codecov / codecov/patch

pkg/index/job/readreplica/rotate/service/rotator.go#L244

Added line #L244 was not covered by tests

View more details on Codecov