Comment non-used function add missing includes

Author: felipesantosk

sdk/include/opentelemetry/sdk/metrics/aggregation/base2_exponential_histogram_aggregation.h

@@ -3,13 +3,16 @@
 
 #pragma once
 
+#include <memory>
+#include <mutex>
+
 #include "opentelemetry/common/spin_lock_mutex.h"
-#include "opentelemetry/sdk/metrics/aggregation/aggregation.h"
 #include "opentelemetry/sdk/metrics/aggregation/aggregation_config.h"
+#include "opentelemetry/sdk/metrics/aggregation/aggregation.h"
 #include "opentelemetry/sdk/metrics/aggregation/base2_exponential_histogram_indexer.h"
-
-#include <memory>
-#include <mutex>
+#include "opentelemetry/sdk/metrics/data/circular_buffer.h"
+#include "opentelemetry/sdk/metrics/data/metric_data.h"
+#include "opentelemetry/sdk/metrics/data/point_data.h"
 
 OPENTELEMETRY_BEGIN_NAMESPACE
 namespace sdk

sdk/include/opentelemetry/sdk/metrics/aggregation/base2_exponential_histogram_indexer.h

@@ -3,9 +3,9 @@
 
 #pragma once
 
-#include "opentelemetry/version.h"
+#include <stdint.h>
 
-#include <cstdint>
+#include "opentelemetry/version.h"
 
 OPENTELEMETRY_BEGIN_NAMESPACE
 namespace sdk

sdk/src/metrics/aggregation/base2_exponential_histogram_aggregation.cc

@@ -37,19 +37,19 @@ uint32_t GetScaleReduction(int32_t start_index, int32_t end_index, size_t max_bu
   return scale_reduction;
 }
 
-uint32_t GetScaleReduction(const AdaptingCircularBufferCounter &first,
-                           const AdaptingCircularBufferCounter &second,
-                           size_t max_buckets)
-{
-  if (first.Empty() || second.Empty())
-  {
-    return 0;
-  }
-
-  const int32_t start_index = std::min(first.StartIndex(), second.StartIndex());
-  const int32_t end_index   = std::max(first.EndIndex(), second.EndIndex());
-  return GetScaleReduction(start_index, end_index, max_buckets);
-}
+// Commented out as it is not used
+// uint32_t GetScaleReduction(const AdaptingCircularBufferCounter &first,
+//                            const AdaptingCircularBufferCounter &second,
+//                            size_t max_buckets)
+// {
+//   if (first.Empty() || second.Empty())
+//   {
+//     return 0;
+//   }
+//   const int32_t start_index = std::min(first.StartIndex(), second.StartIndex());
+//   const int32_t end_index   = std::max(first.EndIndex(), second.EndIndex());
+//   return GetScaleReduction(start_index, end_index, max_buckets);
+// }
 
 void DownscaleBuckets(AdaptingCircularBufferCounter *buckets, uint32_t by) noexcept
 {