General cleanup.

- added references to yaml schema
- simplified propagators

Author: marcalff

sdk/include/opentelemetry/sdk/configuration/aggregation_configuration.h

@@ -13,6 +13,8 @@ namespace sdk
 namespace configuration
 {
 
+// YAML-SCHEMA: schema/meter_provider.json
+// YAML-NODE: aggregation
 class AggregationConfiguration
 {
 public:

sdk/include/opentelemetry/sdk/configuration/always_off_sampler_configuration.h

@@ -13,6 +13,8 @@ namespace sdk
 namespace configuration
 {
 
+// YAML-SCHEMA: schema/tracer_provider.json
+// YAML-NODE: always_off
 class AlwaysOffSamplerConfiguration : public SamplerConfiguration
 {
 public:

sdk/include/opentelemetry/sdk/configuration/always_on_sampler_configuration.h

@@ -13,6 +13,8 @@ namespace sdk
 namespace configuration
 {
 
+// YAML-SCHEMA: schema/tracer_provider.json
+// YAML-NODE: always_on
 class AlwaysOnSamplerConfiguration : public SamplerConfiguration
 {
 public:

sdk/include/opentelemetry/sdk/configuration/attribute_limit_configuration.h renamed to sdk/include/opentelemetry/sdk/configuration/attribute_limits_configuration.h

@@ -13,8 +13,9 @@ namespace sdk
 namespace configuration
 {
 
-// REF: schema/opentelemetry_configuration.json
-class AttributeLimitConfiguration
+// YAML-SCHEMA: schema/opentelemetry_configuration.json
+// YAML-NODE: AttributeLimits
+class AttributeLimitsConfiguration
 {
 public:
   size_t attribute_value_length_limit;

sdk/include/opentelemetry/sdk/configuration/attributes_configuration.h

@@ -13,6 +13,9 @@ namespace sdk
 namespace configuration
 {
 
+// YAML-SCHEMA: schema/resource.json
+// YAML-NODE: AttributeNameValue
+// FIXME: Name/Value/Type
 class AttributesConfiguration
 {
 public:

sdk/include/opentelemetry/sdk/configuration/base2_exponential_bucket_histogram_aggregation_configuration.h

@@ -14,6 +14,8 @@ namespace sdk
 namespace configuration
 {
 
+// YAML-SCHEMA: schema/meter_provider.json
+// YAML-NODE: base2_exponential_bucket_histogram
 class Base2ExponentialBucketHistogramAggregationConfiguration : public AggregationConfiguration
 {
 public:

sdk/include/opentelemetry/sdk/configuration/batch_log_record_processor_configuration.h

@@ -14,7 +14,8 @@ namespace sdk
 namespace configuration
 {
 
-// REF: schema/logger_provider.json
+// YAML-SCHEMA: schema/logger_provider.json
+// YAML-NODE: BatchLogRecordProcessor
 class BatchLogRecordProcessorConfiguration : public LogRecordProcessorConfiguration
 {
 public:

sdk/include/opentelemetry/sdk/configuration/batch_span_processor_configuration.h

@@ -14,6 +14,8 @@ namespace sdk
 namespace configuration
 {
 
+// YAML-SCHEMA: schema/tracer_provider.json
+// YAML-NODE: BatchSpanProcessor
 class BatchSpanProcessorConfiguration : public SpanProcessorConfiguration
 {
 public:

sdk/include/opentelemetry/sdk/configuration/composite_propagator_configuration.h

@@ -5,7 +5,7 @@
 
 #include <string>
 
-#include "opentelemetry/sdk/configuration/attribute_limit_configuration.h"
+#include "opentelemetry/sdk/configuration/attribute_limits_configuration.h"
 #include "opentelemetry/sdk/configuration/document.h"
 #include "opentelemetry/sdk/configuration/logger_provider_configuration.h"
 #include "opentelemetry/sdk/configuration/meter_provider_configuration.h"
@@ -14,13 +14,35 @@
 #include "opentelemetry/sdk/configuration/tracer_provider_configuration.h"
 #include "opentelemetry/version.h"
 
+/*
+ * General notes about configuration classes.
+ *
+ * Each Yaml node that exists in the yaml schema,
+ * as defined by https://github.com/open-telemetry/opentelemetry-configuration
+ * is represented by a C++ class.
+ * Special comments are used to relate the C++ class to
+ * the Yaml node it represents.
+ *
+ * YAML-SCHEMA: points to the relevant file within the
+ * opentelemetry-configuration repository
+ * YAML-NODE: points to the relevant node within the file.
+ *
+ * For example,
+ *   C++ class opentelemetry::sdk::configuration::Configuration
+ * corresponds to
+ *   Yaml node OpenTelemetryConfiguration,
+ * in file
+ * https://github.com/open-telemetry/opentelemetry-configuration/blob/main/schema/opentelemetry_configuration.json
+ */
+
 OPENTELEMETRY_BEGIN_NAMESPACE
 namespace sdk
 {
 namespace configuration
 {
 
-// REF: schema/opentelemetry_configuration.json
+// YAML-SCHEMA: schema/opentelemetry_configuration.json
+// YAML-NODE: OpenTelemetryConfiguration
 class Configuration
 {
 public:
@@ -30,7 +52,7 @@ class Configuration
   std::string file_format;
   bool disabled{false};
 
-  std::unique_ptr<AttributeLimitConfiguration> attribute_limits;
+  std::unique_ptr<AttributeLimitsConfiguration> attribute_limits;
   std::unique_ptr<LoggerProviderConfiguration> logger_provider;
   std::unique_ptr<MeterProviderConfiguration> meter_provider;
   std::unique_ptr<PropagatorConfiguration> propagator;