when report-client failed to serialize metrics, throwing errors rathe…

…r than ignore
chengjoey · May 24, 2024 · c3a9aec · c3a9aec
1 parent 8f6b0b3
commit c3a9aec
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/internal/pkg/metrics/report/collector.go b/internal/pkg/metrics/report/collector.go
@@ -70,7 +70,7 @@ func (c *ReportClient) Send(in []*Metric) error {
 		}
 		requestBuffer, err := c.serialize(group)
 		if err != nil {
-			continue
+			return fmt.Errorf("failed to serialize metrics, name: %s, error: %v", group.Name, err)
 		}
 		for i := 0; i < c.CFG.ReportConfig.Collector.Retry; i++ {
 			if err = c.write(group.Name, requestBuffer); err == nil {