fix(discover): avoid defer accumulation in stream loop (#1422)

* fix(discover): avoid defer accumulation in stream loop

* fix(discover): only report statistics for normal responses

Author: daihao4371

apiserver/grpcserver/discover/v1/client_access.go

@@ -131,18 +131,6 @@ func (g *DiscoverServer) Discover(server apiservice.PolarisGRPC_DiscoverServer)
 		var out *apiservice.DiscoverResponse
 		var action string
 		startTime := commontime.CurrentMillisecond()
-		defer func() {
-			plugin.GetStatis().ReportDiscoverCall(metrics.ClientDiscoverMetric{
-				Action:    action,
-				ClientIP:  utils.ParseClientAddress(ctx),
-				Namespace: in.GetService().GetNamespace().GetValue(),
-				Resource:  in.GetType().String() + ":" + in.GetService().GetName().GetValue(),
-				Timestamp: startTime,
-				CostTime:  commontime.CurrentMillisecond() - startTime,
-				Revision:  out.GetService().GetRevision().GetValue(),
-				Success:   out.GetCode().GetValue() > uint32(apimodel.Code_DataNoChange),
-			})
-		}()
 
 		// 兼容。如果请求中带了token，优先使用该token
 		if in.GetService().GetToken().GetValue() != "" {
@@ -176,6 +164,17 @@ func (g *DiscoverServer) Discover(server apiservice.PolarisGRPC_DiscoverServer)
 		if err != nil {
 			return err
 		}
+
+		plugin.GetStatis().ReportDiscoverCall(metrics.ClientDiscoverMetric{
+			Action:    action,
+			ClientIP:  utils.ParseClientAddress(ctx),
+			Namespace: in.GetService().GetNamespace().GetValue(),
+			Resource:  in.GetType().String() + ":" + in.GetService().GetName().GetValue(),
+			Timestamp: startTime,
+			CostTime:  commontime.CurrentMillisecond() - startTime,
+			Revision:  out.GetService().GetRevision().GetValue(),
+			Success:   out.GetCode().GetValue() > uint32(apimodel.Code_DataNoChange),
+		})
 	}
 }