polish: pr feedback and correcting the build.

vasantteja · vasantteja · commit c6b6dcc4c566 · 2025-04-07T22:36:33.000-04:00
diff --git a/docs/apidiffs/current_vs_latest/opentelemetry-exporter-common.txt b/docs/apidiffs/current_vs_latest/opentelemetry-exporter-common.txt
@@ -1,2 +1,2 @@
-Comparing source compatibility of opentelemetry-exporter-common-1.49.0-SNAPSHOT.jar against opentelemetry-exporter-common-1.48.0.jar
+Comparing source compatibility of opentelemetry-exporter-common-1.49.0-SNAPSHOT.jar against opentelemetry-exporter-common-1.49.0.jar
 No changes.
diff --git a/docs/apidiffs/current_vs_latest/opentelemetry-exporter-otlp.txt b/docs/apidiffs/current_vs_latest/opentelemetry-exporter-otlp.txt
@@ -1,19 +1,2 @@
-Comparing source compatibility of opentelemetry-exporter-otlp-1.49.0-SNAPSHOT.jar against opentelemetry-exporter-otlp-1.48.0.jar
-***  MODIFIED CLASS: PUBLIC FINAL io.opentelemetry.exporter.otlp.http.logs.OtlpHttpLogRecordExporterBuilder  (not serializable)
-	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
-	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.exporter.otlp.http.logs.OtlpHttpLogRecordExporterBuilder setExecutorService(java.util.concurrent.ExecutorService)
-***  MODIFIED CLASS: PUBLIC FINAL io.opentelemetry.exporter.otlp.http.metrics.OtlpHttpMetricExporterBuilder  (not serializable)
-	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
-	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.exporter.otlp.http.metrics.OtlpHttpMetricExporterBuilder setExecutorService(java.util.concurrent.ExecutorService)
-***  MODIFIED CLASS: PUBLIC FINAL io.opentelemetry.exporter.otlp.http.trace.OtlpHttpSpanExporterBuilder  (not serializable)
-	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
-	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.exporter.otlp.http.trace.OtlpHttpSpanExporterBuilder setExecutorService(java.util.concurrent.ExecutorService)
-***  MODIFIED CLASS: PUBLIC FINAL io.opentelemetry.exporter.otlp.logs.OtlpGrpcLogRecordExporterBuilder  (not serializable)
-	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
-	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.exporter.otlp.logs.OtlpGrpcLogRecordExporterBuilder setExecutorService(java.util.concurrent.ExecutorService)
-***  MODIFIED CLASS: PUBLIC FINAL io.opentelemetry.exporter.otlp.metrics.OtlpGrpcMetricExporterBuilder  (not serializable)
-	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
-	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.exporter.otlp.metrics.OtlpGrpcMetricExporterBuilder setExecutorService(java.util.concurrent.ExecutorService)
-***  MODIFIED CLASS: PUBLIC FINAL io.opentelemetry.exporter.otlp.trace.OtlpGrpcSpanExporterBuilder  (not serializable)
-	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
-	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.exporter.otlp.trace.OtlpGrpcSpanExporterBuilder setExecutorService(java.util.concurrent.ExecutorService)
+Comparing source compatibility of opentelemetry-exporter-otlp-1.49.0-SNAPSHOT.jar against opentelemetry-exporter-otlp-1.49.0.jar
+No changes.
diff --git a/docs/apidiffs/current_vs_latest/opentelemetry-exporter-sender-grpc-managed-channel.txt b/docs/apidiffs/current_vs_latest/opentelemetry-exporter-sender-grpc-managed-channel.txt
@@ -1,2 +1,2 @@
-Comparing source compatibility of opentelemetry-exporter-sender-grpc-managed-channel-1.49.0-SNAPSHOT.jar against opentelemetry-exporter-sender-grpc-managed-channel-1.48.0.jar
+Comparing source compatibility of opentelemetry-exporter-sender-grpc-managed-channel-1.49.0-SNAPSHOT.jar against opentelemetry-exporter-sender-grpc-managed-channel-1.49.0.jar
 No changes.
diff --git a/docs/apidiffs/current_vs_latest/opentelemetry-exporter-sender-jdk.txt b/docs/apidiffs/current_vs_latest/opentelemetry-exporter-sender-jdk.txt
@@ -1,2 +1,2 @@
-Comparing source compatibility of opentelemetry-exporter-sender-jdk-1.49.0-SNAPSHOT.jar against opentelemetry-exporter-sender-jdk-1.48.0.jar
+Comparing source compatibility of opentelemetry-exporter-sender-jdk-1.49.0-SNAPSHOT.jar against opentelemetry-exporter-sender-jdk-1.49.0.jar
 No changes.
diff --git a/docs/apidiffs/current_vs_latest/opentelemetry-exporter-sender-okhttp.txt b/docs/apidiffs/current_vs_latest/opentelemetry-exporter-sender-okhttp.txt
@@ -1,2 +1,2 @@
-Comparing source compatibility of opentelemetry-exporter-sender-okhttp-1.49.0-SNAPSHOT.jar against opentelemetry-exporter-sender-okhttp-1.48.0.jar
+Comparing source compatibility of opentelemetry-exporter-sender-okhttp-1.49.0-SNAPSHOT.jar against opentelemetry-exporter-sender-okhttp-1.49.0.jar
 No changes.
diff --git a/docs/apidiffs/current_vs_latest/opentelemetry-opentracing-shim.txt b/docs/apidiffs/current_vs_latest/opentelemetry-opentracing-shim.txt
@@ -1,2 +1,2 @@
-Comparing source compatibility of opentelemetry-opentracing-shim-1.49.0-SNAPSHOT.jar against opentelemetry-opentracing-shim-1.48.0.jar
+Comparing source compatibility of opentelemetry-opentracing-shim-1.49.0-SNAPSHOT.jar against opentelemetry-opentracing-shim-1.49.0.jar
 No changes.
diff --git a/docs/apidiffs/current_vs_latest/opentelemetry-sdk-common.txt b/docs/apidiffs/current_vs_latest/opentelemetry-sdk-common.txt
@@ -1,2 +1,2 @@
-Comparing source compatibility of opentelemetry-sdk-common-1.49.0-SNAPSHOT.jar against opentelemetry-sdk-common-1.48.0.jar
+Comparing source compatibility of opentelemetry-sdk-common-1.49.0-SNAPSHOT.jar against opentelemetry-sdk-common-1.49.0.jar
 No changes.
diff --git a/docs/apidiffs/current_vs_latest/opentelemetry-sdk-metrics.txt b/docs/apidiffs/current_vs_latest/opentelemetry-sdk-metrics.txt
@@ -1,4 +1,4 @@
-Comparing source compatibility of opentelemetry-sdk-metrics-1.49.0-SNAPSHOT.jar against opentelemetry-sdk-metrics-1.48.0.jar
+Comparing source compatibility of opentelemetry-sdk-metrics-1.49.0-SNAPSHOT.jar against opentelemetry-sdk-metrics-1.49.0.jar
 +++  NEW CLASS: PUBLIC(+) ABSTRACT(+) io.opentelemetry.sdk.metrics.data.DelegatingMetricData  (not serializable)
 	+++  CLASS FILE FORMAT VERSION: 52.0 <- n.a.
 	+++  NEW INTERFACE: io.opentelemetry.sdk.metrics.data.MetricData
@@ -18,7 +18,7 @@ Comparing source compatibility of opentelemetry-sdk-metrics-1.49.0-SNAPSHOT.jar
 	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.metrics.data.DoubleExemplarData create(io.opentelemetry.api.common.Attributes, long, io.opentelemetry.api.trace.SpanContext, double)
 ***  MODIFIED INTERFACE: PUBLIC ABSTRACT io.opentelemetry.sdk.metrics.data.DoublePointData  (not serializable)
 	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
-	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.metrics.data.DoublePointData create(long, long, io.opentelemetry.api.common.Attributes, double)
+	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.metrics.data.DoublePointData create(long, long, io.opentelemetry.api.common.Attributes, double, java.util.List<io.opentelemetry.sdk.metrics.data.DoubleExemplarData>)
 ***  MODIFIED INTERFACE: PUBLIC ABSTRACT io.opentelemetry.sdk.metrics.data.ExponentialHistogramBuckets  (not serializable)
 	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
 	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.metrics.data.ExponentialHistogramBuckets create(int, int, java.util.List<java.lang.Long>)
@@ -31,8 +31,8 @@ Comparing source compatibility of opentelemetry-sdk-metrics-1.49.0-SNAPSHOT.jar
 ***  MODIFIED INTERFACE: PUBLIC ABSTRACT io.opentelemetry.sdk.metrics.data.GaugeData  (not serializable)
 	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
 	GENERIC TEMPLATES: === T:io.opentelemetry.sdk.metrics.data.PointData
-	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.metrics.data.GaugeData<T> create(java.util.Collection<T>)
-		GENERIC TEMPLATES: +++ T:io.opentelemetry.sdk.metrics.data.PointData
+	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.metrics.data.GaugeData<io.opentelemetry.sdk.metrics.data.DoublePointData> createDoubleGaugeData(java.util.Collection<io.opentelemetry.sdk.metrics.data.DoublePointData>)
+	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.metrics.data.GaugeData<io.opentelemetry.sdk.metrics.data.LongPointData> createLongGaugeData(java.util.Collection<io.opentelemetry.sdk.metrics.data.LongPointData>)
 ***  MODIFIED INTERFACE: PUBLIC ABSTRACT io.opentelemetry.sdk.metrics.data.HistogramData  (not serializable)
 	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
 	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.metrics.data.HistogramData create(io.opentelemetry.sdk.metrics.data.AggregationTemporality, java.util.Collection<io.opentelemetry.sdk.metrics.data.HistogramPointData>)
@@ -48,8 +48,8 @@ Comparing source compatibility of opentelemetry-sdk-metrics-1.49.0-SNAPSHOT.jar
 ***  MODIFIED INTERFACE: PUBLIC ABSTRACT io.opentelemetry.sdk.metrics.data.SumData  (not serializable)
 	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
 	GENERIC TEMPLATES: === T:io.opentelemetry.sdk.metrics.data.PointData
-	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.metrics.internal.data.ImmutableSumData<T> create(boolean, io.opentelemetry.sdk.metrics.data.AggregationTemporality, java.util.Collection<T>)
-		GENERIC TEMPLATES: +++ T:io.opentelemetry.sdk.metrics.data.PointData
+	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.metrics.data.SumData<io.opentelemetry.sdk.metrics.data.DoublePointData> createDoubleSumData(boolean, io.opentelemetry.sdk.metrics.data.AggregationTemporality, java.util.Collection<io.opentelemetry.sdk.metrics.data.DoublePointData>)
+	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.metrics.data.SumData<io.opentelemetry.sdk.metrics.data.LongPointData> createLongSumData(boolean, io.opentelemetry.sdk.metrics.data.AggregationTemporality, java.util.Collection<io.opentelemetry.sdk.metrics.data.LongPointData>)
 ***  MODIFIED INTERFACE: PUBLIC ABSTRACT io.opentelemetry.sdk.metrics.data.SummaryData  (not serializable)
 	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
 	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.metrics.data.SummaryData create(java.util.Collection<io.opentelemetry.sdk.metrics.data.SummaryPointData>)
diff --git a/docs/apidiffs/current_vs_latest/opentelemetry-sdk-trace.txt b/docs/apidiffs/current_vs_latest/opentelemetry-sdk-trace.txt
@@ -1,2 +1,2 @@
-Comparing source compatibility of opentelemetry-sdk-trace-1.49.0-SNAPSHOT.jar against opentelemetry-sdk-trace-1.48.0.jar
+Comparing source compatibility of opentelemetry-sdk-trace-1.49.0-SNAPSHOT.jar against opentelemetry-sdk-trace-1.49.0.jar
 No changes.
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/DelegatingMetricData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/DelegatingMetricData.java
@@ -34,7 +34,6 @@
  * //   }
  * // }
  * }</pre>
- *
  */
 public abstract class DelegatingMetricData implements MetricData {
 
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/DoubleExemplarData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/DoubleExemplarData.java
@@ -18,15 +18,6 @@
 @Immutable
 public interface DoubleExemplarData extends ExemplarData {
 
-  /**
-   * Construct a new exemplar.
-   *
-   * @param filteredAttributes The set of {@link Attributes} not already associated with the {@link
-   *     PointData}.
-   * @param recordTimeNanos The time when the sample qas recorded in nanoseconds.
-   * @param spanContext The associated span context.
-   * @param value The value recorded.
-   */
   static DoubleExemplarData create(
       Attributes filteredAttributes, long recordTimeNanos, SpanContext spanContext, double value) {
     return ImmutableDoubleExemplarData.create(
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/DoublePointData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/DoublePointData.java
@@ -7,7 +7,6 @@
 
 import io.opentelemetry.api.common.Attributes;
 import io.opentelemetry.sdk.metrics.internal.data.ImmutableDoublePointData;
-import java.util.Collections;
 import java.util.List;
 
 /**
@@ -17,20 +16,14 @@
  */
 public interface DoublePointData extends PointData {
 
-  /**
-   * Creates a {@link DoublePointData}.
-   *
-   * @param startEpochNanos The starting time for the period where this point was sampled. Note:
-   *     While start time is optional in OTLP, all SDKs should produce it for all their metrics, so
-   *     it is required here.
-   * @param epochNanos The ending time for the period when this value was sampled.
-   * @param attributes The set of attributes associated with this point.
-   * @param value The value that was sampled.
-   */
   static DoublePointData create(
-      long startEpochNanos, long epochNanos, Attributes attributes, double value) {
+      long startEpochNanos,
+      long epochNanos,
+      Attributes attributes,
+      double value,
+      List<DoubleExemplarData> exemplars) {
     return ImmutableDoublePointData.create(
-        startEpochNanos, epochNanos, attributes, value, Collections.emptyList());
+        startEpochNanos, epochNanos, attributes, value, exemplars);
   }
 
   /** Returns the value of the data point. */
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/ExponentialHistogramBuckets.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/ExponentialHistogramBuckets.java
@@ -31,7 +31,6 @@ public interface ExponentialHistogramBuckets {
    *
    * @return a ExponentialHistogramBuckets.
    */
-  @SuppressWarnings("TooManyParameters")
   static ExponentialHistogramBuckets create(int scale, int offset, List<Long> bucketCounts) {
     return ImmutableExponentialHistogramBuckets.create(scale, offset, bucketCounts);
   }
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/ExponentialHistogramData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/ExponentialHistogramData.java
@@ -25,7 +25,6 @@
 @Immutable
 public interface ExponentialHistogramData extends Data<ExponentialHistogramPointData> {
 
-  /** Returns a new {@link ExponentialHistogramData}. */
   static ExponentialHistogramData create(
       AggregationTemporality temporality, Collection<ExponentialHistogramPointData> points) {
     return ImmutableExponentialHistogramData.create(temporality, points);
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/ExponentialHistogramPointData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/ExponentialHistogramPointData.java
@@ -26,11 +26,6 @@
 @Immutable
 public interface ExponentialHistogramPointData extends PointData {
 
-  /**
-   * Create a ExponentialHistogramPointData.
-   *
-   * @return a ExponentialHistogramPointData.
-   */
   @SuppressWarnings("TooManyParameters")
   static ExponentialHistogramPointData create(
       int scale,
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/GaugeData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/GaugeData.java
@@ -17,14 +17,12 @@
 @Immutable
 public interface GaugeData<T extends PointData> extends Data<T> {
 
-  /**
-   * Creates an instance of {@link GaugeData} with the given collection of points.
-   *
-   * @param <T> the type of the point data
-   * @param points the collection of points to be included in the gauge data
-   * @return an instance of {@link ImmutableGaugeData} containing the provided points
-   */
-  static <T extends PointData> GaugeData<T> create(Collection<T> points) {
+  // For double gauges
+  static GaugeData<DoublePointData> createDoubleGaugeData(Collection<DoublePointData> points) {
+    return ImmutableGaugeData.create(points);
+  }
+
+  static GaugeData<LongPointData> createLongGaugeData(Collection<LongPointData> points) {
     return ImmutableGaugeData.create(points);
   }
 }
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/HistogramData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/HistogramData.java
@@ -17,13 +17,6 @@
 @Immutable
 public interface HistogramData extends Data<HistogramPointData> {
 
-  /**
-   * Creates a new instance of {@link HistogramData}.
-   *
-   * @param temporality the aggregation temporality of the histogram data
-   * @param points the collection of histogram point data
-   * @return a new instance of {@link HistogramData}
-   */
   static HistogramData create(
       AggregationTemporality temporality, Collection<HistogramPointData> points) {
     return ImmutableHistogramData.create(temporality, points);
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/HistogramPointData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/HistogramPointData.java
@@ -19,13 +19,6 @@
 @Immutable
 public interface HistogramPointData extends PointData {
 
-  /**
-   * Creates a HistogramPointData. For a Histogram with N defined boundaries, there should be N+1
-   * counts.
-   *
-   * @return a HistogramPointData.
-   * @throws IllegalArgumentException if the given boundaries/counts were invalid
-   */
   @SuppressWarnings("TooManyParameters")
   static HistogramPointData create(
       long startEpochNanos,
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/LongExemplarData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/LongExemplarData.java
@@ -18,15 +18,6 @@
 @Immutable
 public interface LongExemplarData extends ExemplarData {
 
-  /**
-   * Construct a new exemplar.
-   *
-   * @param filteredAttributes The set of {@link Attributes} not already associated with the {@link
-   *     PointData}.
-   * @param recordTimeNanos The time when the sample qas recorded in nanoseconds.
-   * @param spanContext The associated span context.
-   * @param value The value recorded.
-   */
   static LongExemplarData create(
       Attributes filteredAttributes, long recordTimeNanos, SpanContext spanContext, long value) {
     return ImmutableLongExemplarData.create(
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/LongPointData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/LongPointData.java
@@ -17,16 +17,6 @@
  */
 public interface LongPointData extends PointData {
 
-  /**
-   * Creates a {@link LongPointData}.
-   *
-   * @param startEpochNanos The starting time for the period where this point was sampled. Note:
-   *     While start time is optional in OTLP, all SDKs should produce it for all their metrics, so
-   *     it is required here.
-   * @param epochNanos The ending time for the period when this value was sampled.
-   * @param attributes The set of attributes associated with this point.
-   * @param value The value that was sampled.
-   */
   static LongPointData create(
       long startEpochNanos, long epochNanos, Attributes attributes, long value) {
     return ImmutableLongPointData.create(
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/SumData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/SumData.java
@@ -17,16 +17,13 @@
 @Immutable
 public interface SumData<T extends PointData> extends Data<T> {
 
-  /**
-   * Creates a new instance of {@link SumData}.
-   *
-   * @param isMonotonic {@code true} if the sum is monotonic.
-   * @param temporality the aggregation temporality of the sum data
-   * @param points the collection of sum point data
-   * @return a new instance of {@link SumData}
-   */
-  static <T extends PointData> ImmutableSumData<T> create(
-      boolean isMonotonic, AggregationTemporality temporality, Collection<T> points) {
+  static SumData<LongPointData> createLongSumData(
+      boolean isMonotonic, AggregationTemporality temporality, Collection<LongPointData> points) {
+    return ImmutableSumData.create(isMonotonic, temporality, points);
+  }
+
+  static SumData<DoublePointData> createDoubleSumData(
+      boolean isMonotonic, AggregationTemporality temporality, Collection<DoublePointData> points) {
     return ImmutableSumData.create(isMonotonic, temporality, points);
   }
 
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/SummaryData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/SummaryData.java
@@ -17,13 +17,6 @@
 @Immutable
 public interface SummaryData extends Data<SummaryPointData> {
 
-  /**
-   * Creates a new instance of {@link SummaryData} with the given collection of {@link
-   * SummaryPointData}.
-   *
-   * @param points a collection of {@link SummaryPointData} to be included in the summary data
-   * @return a new instance of {@link SummaryData} containing the provided points
-   */
   static SummaryData create(Collection<SummaryPointData> points) {
     return ImmutableSummaryData.create(points);
   }
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/SummaryPointData.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/SummaryPointData.java
@@ -18,17 +18,6 @@
 @Immutable
 public interface SummaryPointData extends PointData {
 
-  /**
-   * Creates a {@link SummaryPointData}.
-   *
-   * @param startEpochNanos (optional) The starting time for the period where this point was
-   *     sampled.
-   * @param epochNanos The ending time for the period when this value was sampled.
-   * @param attributes The set of attributes associated with this point.
-   * @param count The number of measurements being sumarized.
-   * @param sum The sum of measuremnts being sumarized.
-   * @param percentileValues Calculations of percentile values from measurements.
-   */
   static SummaryPointData create(
       long startEpochNanos,
       long epochNanos,
diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/ValueAtQuantile.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/data/ValueAtQuantile.java
@@ -16,13 +16,6 @@
 @Immutable
 public interface ValueAtQuantile {
 
-  /**
-   * Creates a new {@code ValueAtQuantile} instance with the specified quantile and value.
-   *
-   * @param quantile the quantile for which the value is being recorded
-   * @param value the value at the specified quantile
-   * @return a new {@code ValueAtQuantile} instance
-   */
   static ValueAtQuantile create(double quantile, double value) {
     return ImmutableValueAtQuantile.create(quantile, value);
   }
diff --git a/sdk/metrics/src/test/java/io/opentelemetry/sdk/metrics/data/DoublePointDataTest.java b/sdk/metrics/src/test/java/io/opentelemetry/sdk/metrics/data/DoublePointDataTest.java
@@ -8,14 +8,16 @@
 import static org.assertj.core.api.Assertions.assertThat;
 
 import io.opentelemetry.api.common.Attributes;
+import java.util.Collections;
 import org.junit.jupiter.api.Test;
 
 public class DoublePointDataTest {
 
   @Test
   void create() {
     DoublePointData pointData =
-        DoublePointData.create(1, 2, Attributes.builder().put("key", "value1").build(), 3);
+        DoublePointData.create(
+            1, 2, Attributes.builder().put("key", "value1").build(), 3, Collections.emptyList());
     assertThat(pointData.getStartEpochNanos()).isEqualTo(1);
     assertThat(pointData.getEpochNanos()).isEqualTo(2);
     assertThat(pointData.getAttributes())
diff --git a/sdk/metrics/src/test/java/io/opentelemetry/sdk/metrics/data/GaugeDataTest.java b/sdk/metrics/src/test/java/io/opentelemetry/sdk/metrics/data/GaugeDataTest.java
@@ -7,14 +7,26 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
+import io.opentelemetry.api.common.Attributes;
 import java.util.Collections;
 import org.junit.jupiter.api.Test;
 
 public class GaugeDataTest {
 
   @Test
-  void create() {
-    GaugeData<LongPointData> gaugeData = GaugeData.create(Collections.emptyList());
-    assertThat(gaugeData.getPoints()).isEmpty();
+  void createLongGaugeData() {
+    LongPointData point = LongPointData.create(0, 0, Attributes.empty(), 1);
+    GaugeData<LongPointData> gaugeData =
+        GaugeData.createLongGaugeData(Collections.singleton(point));
+    assertThat(gaugeData.getPoints()).containsExactly(point);
+  }
+
+  @Test
+  void createDoubleGaugeData() {
+    DoublePointData point =
+        DoublePointData.create(0, 0, Attributes.empty(), 1.0, Collections.emptyList());
+    GaugeData<DoublePointData> gaugeData =
+        GaugeData.createDoubleGaugeData(Collections.singleton(point));
+    assertThat(gaugeData.getPoints()).containsExactly(point);
   }
 }
diff --git a/sdk/metrics/src/test/java/io/opentelemetry/sdk/metrics/data/SumDataTest.java b/sdk/metrics/src/test/java/io/opentelemetry/sdk/metrics/data/SumDataTest.java
diff --git a/sdk/testing/src/main/java/io/opentelemetry/sdk/testing/metrics/TestMetricData.java b/sdk/testing/src/main/java/io/opentelemetry/sdk/testing/metrics/TestMetricData.java

Original file line number	Diff line number	Diff line change
`@@ -1,2 +1,2 @@`
`1`		`-Comparing source compatibility of opentelemetry-exporter-common-1.49.0-SNAPSHOT.jar against opentelemetry-exporter-common-1.48.0.jar`
	`1`	`+Comparing source compatibility of opentelemetry-exporter-common-1.49.0-SNAPSHOT.jar against opentelemetry-exporter-common-1.49.0.jar`
`2`	`2`	`No changes.`