Skip to content
revive

修复分布式限流场景下rpc连接泄漏的问题 #322

Sign in to view logs

修复分布式限流场景下rpc连接泄漏的问题

修复分布式限流场景下rpc连接泄漏的问题 #322

Re-run triggered February 10, 2025 02:14
@sak0sak0
#232
captainroy-hy:royyuewang/fix/ratelimit-grpc-leaking
Status Success
Total duration 40s
Artifacts –

revive.yml

on: pull_request
Matrix: Run Revive Action
Fit to window
Zoom out
Zoom in

Annotations

60 warnings
Run Revive Action (1.15.x): pkg/model/pb/metric/code.go#L28
exported const ExecuteSuccess should have comment (or a comment on this block) or be unexported
Run Revive Action (1.15.x): pkg/model/pb/metric/v2/code.go#L18
should have a package comment
Run Revive Action (1.15.x): pkg/model/pb/metric/code.go#L18
should have a package comment
Run Revive Action (1.15.x): examples/circuitbreaker/instance/consumer/main.go#L100
parameter 'r' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.15.x): examples/circuitbreaker/interface/consumer/main.go#L75
parameter 'ctx' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.15.x): benchmark/benchsimple/main.go#L71
var maxQps should be maxQPS
Run Revive Action (1.15.x): examples/configuration/group/main.go#L18
should have a package comment
Run Revive Action (1.15.x): pkg/plugin/loadbalancer/util.go#L25
parameter 'ctx' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.15.x): examples/mock/main.go#L1
invalid file examples/mock/main.go: examples/mock/main.go:69:13: expected ';', found 0 (and 1 more errors)
Run Revive Action (1.15.x): examples/quickstart/consumer/main.go#L74
parameter 'r' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.18.x): pkg/plugin/circuitbreaker/proxy.go#L32
comment on exported method Proxy.CheckResource should be of the form "CheckResource ..."
Run Revive Action (1.18.x): pkg/plugin/circuitbreaker/proxy.go#L37
comment on exported method Proxy.Report should be of the form "Report ..."
Run Revive Action (1.18.x): pkg/plugin/circuitbreaker/circuitbreaker.go#L18
should have a package comment
Run Revive Action (1.18.x): pkg/plugin/common/plugin.go#L18
should have a package comment
Run Revive Action (1.18.x): examples/activehealthcheck/main.go#L18
should have a package comment
Run Revive Action (1.18.x): examples/route/nearby/provider/main.go#L18
should have a package comment
Run Revive Action (1.18.x): examples/configuration/normal/main.go#L18
should have a package comment
Run Revive Action (1.18.x): pkg/plugin/configfilter/configfilter.go#L19
should have a package comment
Run Revive Action (1.18.x): examples/watch/instance/notify/main.go#L76
exported type TestListener should have comment or be unexported
Run Revive Action (1.18.x): pkg/model/pb/metric/code.go#L28
exported const ExecuteSuccess should have comment (or a comment on this block) or be unexported
Run Revive Action (1.17.x): plugin/loadbalancer/common/half_open.go#L18
should have a package comment
Run Revive Action (1.17.x): plugin/healthcheck/udp/config.go#L18
should have a package comment
Run Revive Action (1.17.x): examples/circuitbreaker/instance/consumer/main.go#L100
parameter 'r' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.17.x): examples/circuitbreaker/instance/consumer/main.go#L18
should have a package comment
Run Revive Action (1.17.x): benchmark/benchmain/main.go#L325
parameter 'pos' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.17.x): benchmark/benchmain/main.go#L325
parameter 'ctx' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.17.x): benchmark/benchmain/main.go#L46
const operationRpcDirect should be operationRPCDirect
Run Revive Action (1.17.x): benchmark/benchmain/main.go#L47
const operationRpcNaming should be operationRPCNaming
Run Revive Action (1.17.x): benchmark/benchmain/main.go#L18
should have a package comment
Run Revive Action (1.17.x): pkg/flow/startup/client_report.go#L123
parameter 'taskKey' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.20.x): examples/configuration/fallback/main.go#L18
should have a package comment
Run Revive Action (1.20.x): examples/watch/service/notify/main.go#L18
should have a package comment
Run Revive Action (1.20.x): examples/watch/service/longpull/main.go#L18
should have a package comment
Run Revive Action (1.20.x): examples/watchinstance/main.go#L18
should have a package comment
Run Revive Action (1.20.x): examples/configuration/group/main.go#L18
should have a package comment
Run Revive Action (1.20.x): pkg/model/local/value.go#L18
should have a package comment
Run Revive Action (1.20.x): examples/circuitbreaker/service/provider/main.go#L75
parameter 'r' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.20.x): pkg/algorithm/rand/scalable_rand_test.go#L51
parameter 't' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.20.x): benchmark/benchmain/main.go#L325
parameter 'pos' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.20.x): pkg/algorithm/match/match.go#L18
should have a package comment
Run Revive Action (1.19.x): examples/configuration/crud/main.go#L18
should have a package comment
Run Revive Action (1.19.x): examples/configuration/encrypt/main.go#L18
should have a package comment
Run Revive Action (1.19.x): examples/circuitbreaker/service/provider/main.go#L17
should have a package comment
Run Revive Action (1.19.x): examples/mock/main.go#L1
invalid file examples/mock/main.go: examples/mock/main.go:69:13: expected ';', found 0 (and 1 more errors)
Run Revive Action (1.19.x): examples/loadbalancer/consumer/main.go#L18
should have a package comment
Run Revive Action (1.19.x): examples/configuration/group/main.go#L18
should have a package comment
Run Revive Action (1.19.x): examples/activehealthcheck/main.go#L18
should have a package comment
Run Revive Action (1.19.x): examples/circuitbreaker/interface/consumer/main.go#L18
should have a package comment
Run Revive Action (1.19.x): examples/circuitbreaker/instance/provider/main.go#L17
should have a package comment
Run Revive Action (1.19.x): examples/configuration/fallback/main.go#L18
should have a package comment
Run Revive Action (1.16.x): examples/watch/service/notify/main.go#L18
should have a package comment
Run Revive Action (1.16.x): examples/watch/service/notify/main.go#L44
exported type TestListener should have comment or be unexported
Run Revive Action (1.16.x): examples/watch/service/notify/main.go#L48
comment on exported method TestListener.OnServicesUpdate should be of the form "OnServicesUpdate ..."
Run Revive Action (1.16.x): examples/watchinstance/main.go#L18
should have a package comment
Run Revive Action (1.16.x): examples/circuitbreaker/service/provider/main.go#L75
parameter 'r' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.16.x): pkg/plugin/ratelimiter/model.go#L18
should have a package comment
Run Revive Action (1.16.x): pkg/plugin/serverconnector/proxy.go#L18
should have a package comment
Run Revive Action (1.16.x): examples/loadbalancer/consumer/main.go#L65
parameter 'r' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.16.x): plugin/loadbalancer/weightedrandom/wr.go#L65
parameter 'ignoreHalfOpen' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.16.x): plugin/loadbalancer/common/half_open.go#L18
should have a package comment