[SDK] Header files cleanup, use forward declarations (open-telemetry#…

…2182)

api/include/opentelemetry/trace/span_context_kv_iterable_view.h

@@ -9,8 +9,10 @@
 
 #include "opentelemetry/common/key_value_iterable_view.h"
 #include "opentelemetry/nostd/function_ref.h"
+#include "opentelemetry/nostd/span.h"
 #include "opentelemetry/nostd/string_view.h"
 #include "opentelemetry/nostd/utility.h"
+#include "opentelemetry/trace/span_context.h"
 #include "opentelemetry/trace/span_context_kv_iterable.h"
 #include "opentelemetry/version.h"
 

examples/batch/main.cc

@@ -2,7 +2,11 @@
 // SPDX-License-Identifier: Apache-2.0
 
 #include "opentelemetry/exporters/ostream/span_exporter_factory.h"
+#include "opentelemetry/sdk/resource/resource.h"
 #include "opentelemetry/sdk/trace/batch_span_processor_factory.h"
+#include "opentelemetry/sdk/trace/batch_span_processor_options.h"
+#include "opentelemetry/sdk/trace/exporter.h"
+#include "opentelemetry/sdk/trace/processor.h"
 #include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 #include "opentelemetry/trace/provider.h"
 

examples/grpc/tracer_common.h

@@ -7,7 +7,10 @@
 #include "opentelemetry/context/propagation/text_map_propagator.h"
 #include "opentelemetry/exporters/ostream/span_exporter_factory.h"
 #include "opentelemetry/nostd/shared_ptr.h"
+#include "opentelemetry/sdk/trace/exporter.h"
+#include "opentelemetry/sdk/trace/processor.h"
 #include "opentelemetry/sdk/trace/simple_processor_factory.h"
+#include "opentelemetry/sdk/trace/tracer_context.h"
 #include "opentelemetry/sdk/trace/tracer_context_factory.h"
 #include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 #include "opentelemetry/trace/propagation/http_trace_context.h"

examples/http/tracer_common.h

@@ -4,7 +4,10 @@
 #pragma once
 
 #include "opentelemetry/exporters/ostream/span_exporter_factory.h"
+#include "opentelemetry/sdk/trace/exporter.h"
+#include "opentelemetry/sdk/trace/processor.h"
 #include "opentelemetry/sdk/trace/simple_processor_factory.h"
+#include "opentelemetry/sdk/trace/tracer_context.h"
 #include "opentelemetry/sdk/trace/tracer_context_factory.h"
 #include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 #include "opentelemetry/trace/provider.h"

examples/jaeger/main.cc

@@ -2,6 +2,8 @@
 // SPDX-License-Identifier: Apache-2.0
 
 #include "opentelemetry/exporters/jaeger/jaeger_exporter_factory.h"
+#include "opentelemetry/sdk/trace/exporter.h"
+#include "opentelemetry/sdk/trace/processor.h"
 #include "opentelemetry/sdk/trace/simple_processor_factory.h"
 #include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 #include "opentelemetry/trace/provider.h"

examples/logs_simple/main.cc

@@ -3,13 +3,16 @@
 
 #ifdef ENABLE_LOGS_PREVIEW
 # include "opentelemetry/exporters/ostream/span_exporter_factory.h"
+# include "opentelemetry/sdk/trace/exporter.h"
+# include "opentelemetry/sdk/trace/processor.h"
 # include "opentelemetry/sdk/trace/simple_processor_factory.h"
 # include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 # include "opentelemetry/trace/provider.h"
 
 # include "opentelemetry/exporters/ostream/log_record_exporter.h"
 # include "opentelemetry/logs/provider.h"
 # include "opentelemetry/sdk/logs/logger_provider_factory.h"
+# include "opentelemetry/sdk/logs/processor.h"
 # include "opentelemetry/sdk/logs/simple_log_record_processor_factory.h"
 
 # ifdef BAZEL_BUILD

examples/multithreaded/main.cc

@@ -3,6 +3,8 @@
 
 #include "opentelemetry/exporters/ostream/span_exporter_factory.h"
 #include "opentelemetry/sdk/resource/resource.h"
+#include "opentelemetry/sdk/trace/exporter.h"
+#include "opentelemetry/sdk/trace/processor.h"
 #include "opentelemetry/sdk/trace/simple_processor_factory.h"
 #include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 #include "opentelemetry/trace/provider.h"

examples/otlp/grpc_log_main.cc

@@ -5,8 +5,12 @@
 # include "opentelemetry/exporters/otlp/otlp_grpc_exporter_factory.h"
 # include "opentelemetry/exporters/otlp/otlp_grpc_log_record_exporter_factory.h"
 # include "opentelemetry/logs/provider.h"
+# include "opentelemetry/sdk/logs/exporter.h"
 # include "opentelemetry/sdk/logs/logger_provider_factory.h"
+# include "opentelemetry/sdk/logs/processor.h"
 # include "opentelemetry/sdk/logs/simple_log_record_processor_factory.h"
+# include "opentelemetry/sdk/trace/exporter.h"
+# include "opentelemetry/sdk/trace/processor.h"
 # include "opentelemetry/sdk/trace/simple_processor_factory.h"
 # include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 # include "opentelemetry/trace/provider.h"

examples/otlp/grpc_main.cc

@@ -2,6 +2,7 @@
 // SPDX-License-Identifier: Apache-2.0
 
 #include "opentelemetry/exporters/otlp/otlp_grpc_exporter_factory.h"
+#include "opentelemetry/sdk/trace/processor.h"
 #include "opentelemetry/sdk/trace/simple_processor_factory.h"
 #include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 #include "opentelemetry/trace/provider.h"

examples/otlp/http_log_main.cc

@@ -8,7 +8,9 @@
 # include "opentelemetry/logs/provider.h"
 # include "opentelemetry/sdk/common/global_log_handler.h"
 # include "opentelemetry/sdk/logs/logger_provider_factory.h"
+# include "opentelemetry/sdk/logs/processor.h"
 # include "opentelemetry/sdk/logs/simple_log_record_processor_factory.h"
+# include "opentelemetry/sdk/trace/processor.h"
 # include "opentelemetry/sdk/trace/simple_processor_factory.h"
 # include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 # include "opentelemetry/trace/provider.h"

examples/otlp/http_main.cc

@@ -4,6 +4,7 @@
 #include "opentelemetry/exporters/otlp/otlp_http_exporter_factory.h"
 #include "opentelemetry/exporters/otlp/otlp_http_exporter_options.h"
 #include "opentelemetry/sdk/common/global_log_handler.h"
+#include "opentelemetry/sdk/trace/processor.h"
 #include "opentelemetry/sdk/trace/simple_processor_factory.h"
 #include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 #include "opentelemetry/trace/provider.h"

examples/simple/main.cc

@@ -2,6 +2,8 @@
 // SPDX-License-Identifier: Apache-2.0
 
 #include "opentelemetry/exporters/ostream/span_exporter_factory.h"
+#include "opentelemetry/sdk/trace/exporter.h"
+#include "opentelemetry/sdk/trace/processor.h"
 #include "opentelemetry/sdk/trace/simple_processor_factory.h"
 #include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 #include "opentelemetry/trace/provider.h"

examples/zipkin/main.cc

@@ -2,6 +2,8 @@
 // SPDX-License-Identifier: Apache-2.0
 
 #include "opentelemetry/exporters/zipkin/zipkin_exporter_factory.h"
+#include "opentelemetry/sdk/resource/resource.h"
+#include "opentelemetry/sdk/trace/processor.h"
 #include "opentelemetry/sdk/trace/simple_processor_factory.h"
 #include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 #include "opentelemetry/trace/provider.h"

exporters/elasticsearch/src/es_log_recordable.cc

@@ -4,6 +4,12 @@
 #ifdef ENABLE_LOGS_PREVIEW
 
 # include "opentelemetry/exporters/elasticsearch/es_log_recordable.h"
+# include "opentelemetry/logs/severity.h"
+# include "opentelemetry/sdk/instrumentationscope/instrumentation_scope.h"
+# include "opentelemetry/sdk/resource/resource.h"
+# include "opentelemetry/trace/span_id.h"
+# include "opentelemetry/trace/trace_flags.h"
+# include "opentelemetry/trace/trace_id.h"
 
 OPENTELEMETRY_BEGIN_NAMESPACE
 namespace exporter

exporters/etw/include/opentelemetry/exporters/etw/etw_tracer.h

@@ -32,6 +32,7 @@
 #include "opentelemetry/trace/trace_id.h"
 #include "opentelemetry/trace/tracer_provider.h"
 
+#include "opentelemetry/sdk/common/empty_attributes.h"
 #include "opentelemetry/sdk/trace/exporter.h"
 #include "opentelemetry/sdk/trace/samplers/always_on.h"
 

exporters/jaeger/src/recordable.cc

@@ -3,6 +3,7 @@
 
 #include "opentelemetry/exporters/jaeger/recordable.h"
 #include "opentelemetry/sdk/common/global_log_handler.h"
+#include "opentelemetry/sdk/resource/resource.h"
 #include "opentelemetry/sdk/resource/semantic_conventions.h"
 
 OPENTELEMETRY_BEGIN_NAMESPACE

exporters/jaeger/test/jaeger_exporter_test.cc

@@ -1,11 +1,12 @@
 // Copyright The OpenTelemetry Authors
 // SPDX-License-Identifier: Apache-2.0
 
-#include <opentelemetry/exporters/jaeger/jaeger_exporter.h>
-#include <memory>
-#include <vector>
+#include "opentelemetry/exporters/jaeger/jaeger_exporter.h"
 #include "opentelemetry/sdk/trace/batch_span_processor.h"
+#include "opentelemetry/sdk/trace/batch_span_processor_options.h"
+#include "opentelemetry/sdk/trace/tracer.h"
 #include "opentelemetry/sdk/trace/tracer_provider.h"
+#include "opentelemetry/trace/span_startoptions.h"
 
 #ifdef BAZEL_BUILD
 # include "exporters/jaeger/src/thrift_sender.h"
@@ -16,6 +17,9 @@
 #include <gtest/gtest.h>
 #include "gmock/gmock.h"
 
+#include <memory>
+#include <vector>
+
 namespace trace = opentelemetry::trace;
 namespace nostd = opentelemetry::nostd;
 namespace sdktrace = opentelemetry::sdk::trace;

exporters/ostream/include/opentelemetry/exporters/ostream/log_record_exporter.h

@@ -4,14 +4,17 @@
 #pragma once
 #ifdef ENABLE_LOGS_PREVIEW
 
+# include "opentelemetry/common/attribute_value.h"
 # include "opentelemetry/common/spin_lock_mutex.h"
 # include "opentelemetry/nostd/type_traits.h"
+# include "opentelemetry/sdk/common/attribute_utils.h"
 # include "opentelemetry/sdk/logs/exporter.h"
 
 # include "opentelemetry/version.h"
 
 # include <iostream>
 # include <sstream>
+# include <unordered_map>
 
 OPENTELEMETRY_BEGIN_NAMESPACE
 namespace exporter

exporters/ostream/include/opentelemetry/exporters/ostream/metric_exporter.h

@@ -5,13 +5,23 @@
 
 #include <iostream>
 #include <string>
+
 #include "opentelemetry/common/spin_lock_mutex.h"
 #include "opentelemetry/sdk/metrics/data/metric_data.h"
+#include "opentelemetry/sdk/metrics/export/metric_producer.h"
 #include "opentelemetry/sdk/metrics/instruments.h"
 #include "opentelemetry/sdk/metrics/push_metric_exporter.h"
 #include "opentelemetry/version.h"
 
 OPENTELEMETRY_BEGIN_NAMESPACE
+namespace sdk
+{
+namespace resource
+{
+class Resource;
+} // namespace resource
+} // namespace sdk
+
 namespace exporter
 {
 namespace metrics

exporters/ostream/src/log_record_exporter.cc

@@ -4,7 +4,9 @@
 #ifdef ENABLE_LOGS_PREVIEW
 # include "opentelemetry/exporters/ostream/log_record_exporter.h"
 # include "opentelemetry/exporters/ostream/common_utils.h"
+# include "opentelemetry/sdk/instrumentationscope/instrumentation_scope.h"
 # include "opentelemetry/sdk/logs/read_write_log_record.h"
+# include "opentelemetry/sdk/resource/resource.h"
 # include "opentelemetry/sdk_config.h"
 
 # include <iostream>

exporters/ostream/src/metric_exporter.cc

@@ -2,15 +2,19 @@
 // SPDX-License-Identifier: Apache-2.0
 
 #include "opentelemetry/exporters/ostream/metric_exporter.h"
-#include <algorithm>
-#include <chrono>
-#include <ctime>
-#include <map>
 #include "opentelemetry/exporters/ostream/common_utils.h"
+#include "opentelemetry/sdk/instrumentationscope/instrumentation_scope.h"
 #include "opentelemetry/sdk/metrics/aggregation/default_aggregation.h"
 #include "opentelemetry/sdk/metrics/aggregation/histogram_aggregation.h"
+#include "opentelemetry/sdk/resource/resource.h"
 #include "opentelemetry/sdk_config.h"
 
+#include <algorithm>
+#include <chrono>
+#include <ctime>
+#include <map>
+#include <mutex>
+
 namespace
 {
 std::string timeToString(opentelemetry::common::SystemTimestamp time_stamp)

exporters/ostream/test/ostream_log_test.cc

@@ -3,15 +3,18 @@
 
 #ifdef ENABLE_LOGS_PREVIEW
 
-# include <array>
 # include "opentelemetry/exporters/ostream/log_record_exporter.h"
 # include "opentelemetry/logs/provider.h"
 # include "opentelemetry/nostd/span.h"
+# include "opentelemetry/sdk/instrumentationscope/instrumentation_scope.h"
 # include "opentelemetry/sdk/logs/logger_provider.h"
 # include "opentelemetry/sdk/logs/read_write_log_record.h"
 # include "opentelemetry/sdk/logs/simple_log_record_processor.h"
+# include "opentelemetry/sdk/version/version.h"
 
 # include <gtest/gtest.h>
+
+# include <array>
 # include <iostream>
 
 namespace sdklogs = opentelemetry::sdk::logs;

exporters/ostream/test/ostream_metric_test.cc

@@ -2,18 +2,20 @@
 // SPDX-License-Identifier: Apache-2.0
 
 #include <gtest/gtest.h>
-#include <memory>
-#include <vector>
-#include "opentelemetry/sdk/metrics/instruments.h"
-#include "opentelemetry/sdk/resource/resource_detector.h"
-#include "opentelemetry/sdk/version/version.h"
 
-#include <iostream>
 #include "opentelemetry/exporters/ostream/metric_exporter.h"
+#include "opentelemetry/sdk/instrumentationscope/instrumentation_scope.h"
 #include "opentelemetry/sdk/metrics/aggregation/default_aggregation.h"
 #include "opentelemetry/sdk/metrics/aggregation/histogram_aggregation.h"
 #include "opentelemetry/sdk/metrics/data/metric_data.h"
+#include "opentelemetry/sdk/metrics/instruments.h"
 #include "opentelemetry/sdk/resource/resource.h"
+#include "opentelemetry/sdk/resource/resource_detector.h"
+#include "opentelemetry/sdk/version/version.h"
+
+#include <iostream>
+#include <memory>
+#include <vector>
 
 namespace metric_sdk = opentelemetry::sdk::metrics;
 namespace nostd = opentelemetry::nostd;

exporters/otlp/src/otlp_metric_utils.cc

@@ -3,6 +3,7 @@
 
 #include "opentelemetry/exporters/otlp/otlp_metric_utils.h"
 #include "opentelemetry/exporters/otlp/otlp_populate_attribute_utils.h"
+#include "opentelemetry/sdk/instrumentationscope/instrumentation_scope.h"
 
 OPENTELEMETRY_BEGIN_NAMESPACE
 

exporters/otlp/test/otlp_grpc_log_record_exporter_test.cc

@@ -19,8 +19,10 @@
 # include "opentelemetry/sdk/logs/batch_log_record_processor.h"
 # include "opentelemetry/sdk/logs/exporter.h"
 # include "opentelemetry/sdk/logs/logger_provider.h"
+# include "opentelemetry/sdk/logs/recordable.h"
 # include "opentelemetry/sdk/resource/resource.h"
 # include "opentelemetry/sdk/trace/exporter.h"
+# include "opentelemetry/sdk/trace/processor.h"
 # include "opentelemetry/sdk/trace/simple_processor_factory.h"
 # include "opentelemetry/sdk/trace/tracer_provider_factory.h"
 # include "opentelemetry/trace/provider.h"

exporters/otlp/test/otlp_http_exporter_test.cc

@@ -17,6 +17,7 @@
 # include "opentelemetry/ext/http/client/http_client_factory.h"
 # include "opentelemetry/ext/http/server/http_server.h"
 # include "opentelemetry/sdk/trace/batch_span_processor.h"
+# include "opentelemetry/sdk/trace/batch_span_processor_options.h"
 # include "opentelemetry/sdk/trace/tracer_provider.h"
 # include "opentelemetry/test_common/ext/http/client/nosend/http_client_nosend.h"
 # include "opentelemetry/trace/provider.h"

exporters/otlp/test/otlp_http_metric_exporter_test.cc

@@ -16,9 +16,11 @@
 #include "opentelemetry/common/key_value_iterable_view.h"
 #include "opentelemetry/ext/http/client/http_client_factory.h"
 #include "opentelemetry/ext/http/server/http_server.h"
+#include "opentelemetry/sdk/instrumentationscope/instrumentation_scope.h"
 #include "opentelemetry/sdk/metrics/aggregation/default_aggregation.h"
 #include "opentelemetry/sdk/metrics/aggregation/histogram_aggregation.h"
 #include "opentelemetry/sdk/metrics/data/metric_data.h"
+#include "opentelemetry/sdk/metrics/export/metric_producer.h"
 #include "opentelemetry/sdk/metrics/instruments.h"
 #include "opentelemetry/sdk/resource/resource.h"
 #include "opentelemetry/test_common/ext/http/client/nosend/http_client_nosend.h"

exporters/otlp/test/otlp_recordable_test.cc

@@ -2,6 +2,7 @@
 // SPDX-License-Identifier: Apache-2.0
 
 #include "opentelemetry/exporters/otlp/otlp_recordable.h"
+#include "opentelemetry/sdk/resource/resource.h"
 
 #if defined(__GNUC__)
 // GCC raises -Wsuggest-override warnings on GTest,

exporters/zipkin/src/recordable.cc

@@ -2,6 +2,7 @@
 // SPDX-License-Identifier: Apache-2.0
 
 #include "opentelemetry/exporters/zipkin/recordable.h"
+#include "opentelemetry/sdk/resource/resource.h"
 #include "opentelemetry/sdk/resource/semantic_conventions.h"
 
 #include <map>

exporters/zipkin/test/zipkin_exporter_test.cc

@@ -4,10 +4,10 @@
 #ifndef HAVE_CPP_STDLIB
 
 # include "opentelemetry/exporters/zipkin/zipkin_exporter.h"
-# include <string>
 # include "opentelemetry/ext/http/client/curl/http_client_curl.h"
 # include "opentelemetry/ext/http/server/http_server.h"
 # include "opentelemetry/sdk/trace/batch_span_processor.h"
+# include "opentelemetry/sdk/trace/batch_span_processor_options.h"
 # include "opentelemetry/sdk/trace/tracer_provider.h"
 # include "opentelemetry/trace/provider.h"
 
@@ -16,6 +16,8 @@
 
 # include "nlohmann/json.hpp"
 
+# include <string>
+
 # if defined(_MSC_VER)
 # include "opentelemetry/sdk/common/env_variables.h"
 using opentelemetry::sdk::common::setenv;

ext/test/w3c_tracecontext_test/main.cc

@@ -6,6 +6,7 @@
 #include "opentelemetry/ext/http/client/curl/http_client_curl.h"
 #include "opentelemetry/ext/http/server/http_server.h"
 #include "opentelemetry/sdk/trace/simple_processor.h"
+#include "opentelemetry/sdk/trace/tracer_context.h"
 #include "opentelemetry/sdk/trace/tracer_provider.h"
 #include "opentelemetry/trace/propagation/http_trace_context.h"
 #include "opentelemetry/trace/provider.h"