cleanup

Author: marcalff

exporters/otlp/include/opentelemetry/exporters/otlp/otlp_builder_utils.h

@@ -3,7 +3,7 @@
 
 #pragma once
 
-#include <memory>
+#include <string>
 
 #include "opentelemetry/exporters/otlp/otlp_environment.h"  // For OtlpHeaders
 #include "opentelemetry/exporters/otlp/otlp_http.h"

exporters/otlp/src/otlp_builder_utils.cc

@@ -21,8 +21,10 @@ HttpRequestContentType OtlpBuilderUtils::ConvertOtlpHttpEncoding(
   {
     case opentelemetry::sdk::configuration::OtlpHttpEncoding::protobuf:
       result = exporter::otlp::HttpRequestContentType::kBinary;
+      break;
     case opentelemetry::sdk::configuration::OtlpHttpEncoding::json:
       result = exporter::otlp::HttpRequestContentType::kJson;
+      break;
   }
 
   return result;
@@ -90,10 +92,13 @@ PreferredAggregationTemporality OtlpBuilderUtils::ConvertTemporalityPreference(
   {
     case opentelemetry::sdk::configuration::TemporalityPreference::cumulative:
       result = exporter::otlp::PreferredAggregationTemporality::kCumulative;
+      break;
     case opentelemetry::sdk::configuration::TemporalityPreference::delta:
       result = exporter::otlp::PreferredAggregationTemporality::kDelta;
+      break;
     case opentelemetry::sdk::configuration::TemporalityPreference::low_memory:
       result = exporter::otlp::PreferredAggregationTemporality::kLowMemory;
+      break;
   }
 
   return result;

exporters/otlp/src/otlp_grpc_log_record_builder.cc

@@ -1,7 +1,9 @@
 // Copyright The OpenTelemetry Authors
 // SPDX-License-Identifier: Apache-2.0
 
+#include <chrono>
 #include <memory>
+#include <string>
 #include <utility>
 
 #include "opentelemetry/exporters/otlp/otlp_builder_utils.h"

exporters/otlp/src/otlp_grpc_push_metric_builder.cc

@@ -1,7 +1,9 @@
 // Copyright The OpenTelemetry Authors
 // SPDX-License-Identifier: Apache-2.0
 
+#include <chrono>
 #include <memory>
+#include <string>
 #include <utility>
 
 #include "opentelemetry/exporters/otlp/otlp_builder_utils.h"

exporters/otlp/src/otlp_grpc_span_builder.cc

@@ -1,7 +1,9 @@
 // Copyright The OpenTelemetry Authors
 // SPDX-License-Identifier: Apache-2.0
 
+#include <chrono>
 #include <memory>
+#include <string>
 #include <utility>
 
 #include "opentelemetry/exporters/otlp/otlp_builder_utils.h"

exporters/otlp/src/otlp_http_log_record_builder.cc

@@ -1,10 +1,13 @@
 // Copyright The OpenTelemetry Authors
 // SPDX-License-Identifier: Apache-2.0
 
+#include <chrono>
 #include <memory>
+#include <string>
 #include <utility>
 
 #include "opentelemetry/exporters/otlp/otlp_builder_utils.h"
+#include "opentelemetry/exporters/otlp/otlp_http.h"
 #include "opentelemetry/exporters/otlp/otlp_http_log_record_builder.h"
 #include "opentelemetry/exporters/otlp/otlp_http_log_record_exporter_factory.h"
 #include "opentelemetry/exporters/otlp/otlp_http_log_record_exporter_options.h"

exporters/otlp/src/otlp_http_push_metric_builder.cc

@@ -1,10 +1,13 @@
 // Copyright The OpenTelemetry Authors
 // SPDX-License-Identifier: Apache-2.0
 
+#include <chrono>
 #include <memory>
+#include <string>
 #include <utility>
 
 #include "opentelemetry/exporters/otlp/otlp_builder_utils.h"
+#include "opentelemetry/exporters/otlp/otlp_http.h"
 #include "opentelemetry/exporters/otlp/otlp_http_metric_exporter_factory.h"
 #include "opentelemetry/exporters/otlp/otlp_http_metric_exporter_options.h"
 #include "opentelemetry/exporters/otlp/otlp_http_push_metric_builder.h"

exporters/otlp/src/otlp_http_span_builder.cc

@@ -7,6 +7,7 @@
 #include <utility>
 
 #include "opentelemetry/exporters/otlp/otlp_builder_utils.h"
+#include "opentelemetry/exporters/otlp/otlp_http.h"
 #include "opentelemetry/exporters/otlp/otlp_http_exporter_factory.h"
 #include "opentelemetry/exporters/otlp/otlp_http_exporter_options.h"
 #include "opentelemetry/exporters/otlp/otlp_http_span_builder.h"

exporters/prometheus/src/prometheus_pull_builder.cc

@@ -2,6 +2,7 @@
 // SPDX-License-Identifier: Apache-2.0
 
 #include <memory>
+#include <string>
 #include <utility>
 
 #include "opentelemetry/exporters/prometheus/exporter_factory.h"