avoid expensive clones in splitCreateTimeSeriesRequest (#299)

census-ecosystem · Apr 6, 2022 · 074d8ef · 074d8ef
1 parent dfce60d
commit 074d8ef
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/stats.go b/stats.go
@@ -639,6 +639,8 @@ var createServiceTimeSeries = func(ctx context.Context, c *monitoring.MetricClie
 func splitCreateTimeSeriesRequest(req *monitoringpb.CreateTimeSeriesRequest) (*monitoringpb.CreateTimeSeriesRequest, *monitoringpb.CreateTimeSeriesRequest) {
 	var serviceReq, nonServiceReq *monitoringpb.CreateTimeSeriesRequest
 	serviceTs, nonServiceTs := splitTimeSeries(req.TimeSeries)
+	// reset timeseries as we just split it to avoid cloning it in the calls below
+	req.TimeSeries = nil
 	if len(serviceTs) > 0 {
 		serviceReq = proto.Clone(req).(*monitoringpb.CreateTimeSeriesRequest)
 		serviceReq.TimeSeries = serviceTs