dynatraceexporter: continue processing data points after an error (#9330

) * dynatraceexporter: continue processing data points after an error * Changelog
open-telemetry · Apr 20, 2022 · 6fe7912 · 6fe7912
1 parent b18bf1a
commit 6fe7912
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -78,6 +78,7 @@
 - `signalfxexporter`: Fix bug to enable timeouts for correlating traces and metrics (#9101)
 - `windowsperfcountersreceiver`: fix exported values being integers instead of doubles (#9138)
 - `prometheusreceiver`: Fix issues with relabelling the `job` and `instance` labels. (#8780)
+- `dynatraceexporter`: Continue processing data points after a serialization error. (#9330)
 
 ## v0.48.0
 

diff --git a/exporter/dynatraceexporter/internal/serialization/serialization.go b/exporter/dynatraceexporter/internal/serialization/serialization.go
@@ -45,7 +45,11 @@ func SerializeMetric(logger *zap.Logger, prefix string, metric pmetric.Metric, d
 			)
 
 			if err != nil {
-				return nil, err
+				logger.Sugar().Warnw("Error serializing gauge data point",
+					"name", metric.Name(),
+					"value-type", dp.ValueType().String(),
+					"error", err,
+				)
 			}
 
 			if line != "" {
@@ -67,7 +71,11 @@ func SerializeMetric(logger *zap.Logger, prefix string, metric pmetric.Metric, d
 			)
 
 			if err != nil {
-				return nil, err
+				logger.Sugar().Warnw("Error serializing sum data point",
+					"name", metric.Name(),
+					"value-type", dp.ValueType().String(),
+					"error", err,
+				)
 			}
 
 			if line != "" {
@@ -88,7 +96,10 @@ func SerializeMetric(logger *zap.Logger, prefix string, metric pmetric.Metric, d
 			)
 
 			if err != nil {
-				return nil, err
+				logger.Sugar().Warnw("Error serializing histogram data point",
+					"name", metric.Name(),
+					"error", err,
+				)
 			}
 
 			if line != "" {