balancer/leastrequest: Add verbosity check around build log (#7467)

zasweq · web-flow · commit 4a26a494081a · 2024-08-01T19:02:14.000-04:00
diff --git a/balancer/leastrequest/leastrequest.go b/balancer/leastrequest/leastrequest.go
@@ -112,7 +112,9 @@ type scWithRPCCount struct {
 }
 
 func (lrb *leastRequestBalancer) Build(info base.PickerBuildInfo) balancer.Picker {
-	logger.Infof("least-request: Build called with info: %v", info)
+	if logger.V(2) {
+		logger.Infof("least-request: Build called with info: %v", info)
+	}
 	if len(info.ReadySCs) == 0 {
 		return base.NewErrPicker(balancer.ErrNoSubConnAvailable)
 	}

Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,9 @@ type scWithRPCCount struct {`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`func (lrb *leastRequestBalancer) Build(info base.PickerBuildInfo) balancer.Picker {`
`115`		`- logger.Infof("least-request: Build called with info: %v", info)`
	`115`	`+ if logger.V(2) {`
	`116`	`+ logger.Infof("least-request: Build called with info: %v", info)`
	`117`	`+ }`
`116`	`118`	`if len(info.ReadySCs) == 0 {`
`117`	`119`	`return base.NewErrPicker(balancer.ErrNoSubConnAvailable)`
`118`	`120`	`}`