Remove the @UnstableApi annotation (open-telemetry#5757)

RashmiRam · May 23, 2022 · aec8311 · aec8311
1 parent bff4999
commit aec8311
Show file tree

Hide file tree

Showing 15 changed files with 0 additions and 68 deletions.
diff --git a/...main/java/io/opentelemetry/instrumentation/api/annotation/support/async/package-info.java b/...main/java/io/opentelemetry/instrumentation/api/annotation/support/async/package-info.java
@@ -2,7 +2,4 @@
  * Provides implementations of strategies for instrumenting methods that return asynchronous and/or
  * reactive values so that the operation can be ended when the asynchronous type completes.
  */
-@UnstableApi
 package io.opentelemetry.instrumentation.api.annotation.support.async;
-
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
diff --git a/...t/src/main/java/io/opentelemetry/instrumentation/api/annotation/support/package-info.java b/...t/src/main/java/io/opentelemetry/instrumentation/api/annotation/support/package-info.java
@@ -1,5 +1,2 @@
 /** Provides utility classes to provide support for instrumenting annotated methods. */
-@UnstableApi
 package io.opentelemetry.instrumentation.api.annotation.support;
-
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
diff --git a/...semconv/src/main/java/io/opentelemetry/instrumentation/api/config/ExperimentalConfig.java b/...semconv/src/main/java/io/opentelemetry/instrumentation/api/config/ExperimentalConfig.java
@@ -5,9 +5,6 @@
 
 package io.opentelemetry.instrumentation.api.config;
 
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
-
-@UnstableApi
 public final class ExperimentalConfig {
 
   private static final ExperimentalConfig instance = new ExperimentalConfig(Config.get());

diff --git a/.../opentelemetry/instrumentation/api/instrumenter/db/DbClientCommonAttributesExtractor.java b/.../opentelemetry/instrumentation/api/instrumenter/db/DbClientCommonAttributesExtractor.java
@@ -9,7 +9,6 @@
 
 import io.opentelemetry.api.common.AttributesBuilder;
 import io.opentelemetry.context.Context;
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
 import io.opentelemetry.instrumentation.api.instrumenter.AttributesExtractor;
 import io.opentelemetry.instrumentation.api.internal.SpanKey;
 import io.opentelemetry.instrumentation.api.internal.SpanKeyProvider;
@@ -47,7 +46,6 @@ public final void onEnd(
    * This method is internal and is hence not for public use. Its API is unstable and can change at
    * any time.
    */
-  @UnstableApi
   @Override
   public SpanKey internalGetSpanKey() {
     return SpanKey.DB_CLIENT;

diff --git a/...io/opentelemetry/instrumentation/api/instrumenter/http/HttpClientAttributesExtractor.java b/...io/opentelemetry/instrumentation/api/instrumenter/http/HttpClientAttributesExtractor.java
@@ -9,7 +9,6 @@
 
 import io.opentelemetry.api.common.AttributesBuilder;
 import io.opentelemetry.context.Context;
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
 import io.opentelemetry.instrumentation.api.internal.SpanKey;
 import io.opentelemetry.instrumentation.api.internal.SpanKeyProvider;
 import io.opentelemetry.semconv.trace.attributes.SemanticAttributes;
@@ -73,7 +72,6 @@ public void onEnd(
    * This method is internal and is hence not for public use. Its API is unstable and can change at
    * any time.
    */
-  @UnstableApi
   @Override
   public SpanKey internalGetSpanKey() {
     return SpanKey.HTTP_CLIENT;

diff --git a/...c/main/java/io/opentelemetry/instrumentation/api/instrumenter/http/HttpClientMetrics.java b/...c/main/java/io/opentelemetry/instrumentation/api/instrumenter/http/HttpClientMetrics.java
@@ -14,7 +14,6 @@
 import io.opentelemetry.api.metrics.Meter;
 import io.opentelemetry.context.Context;
 import io.opentelemetry.context.ContextKey;
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
 import io.opentelemetry.instrumentation.api.instrumenter.RequestListener;
 import io.opentelemetry.instrumentation.api.instrumenter.RequestMetrics;
 import java.util.concurrent.TimeUnit;
@@ -25,7 +24,6 @@
  * href="https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/semantic_conventions/http-metrics.md#http-client">HTTP
  * client metrics</a>.
  */
-@UnstableApi
 public final class HttpClientMetrics implements RequestListener {
 
   private static final double NANOS_PER_MS = TimeUnit.MILLISECONDS.toNanos(1);
@@ -40,7 +38,6 @@ public final class HttpClientMetrics implements RequestListener {
    * HttpClientMetrics} on an {@link
    * io.opentelemetry.instrumentation.api.instrumenter.InstrumenterBuilder}.
    */
-  @UnstableApi
   public static RequestMetrics get() {
     return HttpClientMetrics::new;
   }

diff --git a/...c/main/java/io/opentelemetry/instrumentation/api/instrumenter/http/HttpServerMetrics.java b/...c/main/java/io/opentelemetry/instrumentation/api/instrumenter/http/HttpServerMetrics.java
@@ -16,7 +16,6 @@
 import io.opentelemetry.api.metrics.Meter;
 import io.opentelemetry.context.Context;
 import io.opentelemetry.context.ContextKey;
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
 import io.opentelemetry.instrumentation.api.instrumenter.RequestListener;
 import io.opentelemetry.instrumentation.api.instrumenter.RequestMetrics;
 import java.util.concurrent.TimeUnit;
@@ -27,7 +26,6 @@
  * href="https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/semantic_conventions/http-metrics.md#http-server">HTTP
  * server metrics</a>.
  */
-@UnstableApi
 public final class HttpServerMetrics implements RequestListener {
 
   private static final double NANOS_PER_MS = TimeUnit.MILLISECONDS.toNanos(1);
@@ -42,7 +40,6 @@ public final class HttpServerMetrics implements RequestListener {
    * HttpServerMetrics} on an {@link
    * io.opentelemetry.instrumentation.api.instrumenter.InstrumenterBuilder}.
    */
-  @UnstableApi
   public static RequestMetrics get() {
     return HttpServerMetrics::new;
   }

diff --git a/...pentelemetry/instrumentation/api/instrumenter/messaging/MessagingAttributesExtractor.java b/...pentelemetry/instrumentation/api/instrumenter/messaging/MessagingAttributesExtractor.java
@@ -11,7 +11,6 @@
 
 import io.opentelemetry.api.common.AttributesBuilder;
 import io.opentelemetry.context.Context;
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
 import io.opentelemetry.instrumentation.api.instrumenter.AttributesExtractor;
 import io.opentelemetry.instrumentation.api.internal.SpanKey;
 import io.opentelemetry.instrumentation.api.internal.SpanKeyProvider;
@@ -96,7 +95,6 @@ public void onEnd(
    * This method is internal and is hence not for public use. Its API is unstable and can change at
    * any time.
    */
-  @UnstableApi
   @Override
   public SpanKey internalGetSpanKey() {
     switch (operation) {

diff --git a/...a/io/opentelemetry/instrumentation/api/instrumenter/rpc/RpcClientAttributesExtractor.java b/...a/io/opentelemetry/instrumentation/api/instrumenter/rpc/RpcClientAttributesExtractor.java
@@ -5,7 +5,6 @@
 
 package io.opentelemetry.instrumentation.api.instrumenter.rpc;
 
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
 import io.opentelemetry.instrumentation.api.internal.SpanKey;
 import io.opentelemetry.instrumentation.api.internal.SpanKeyProvider;
 
@@ -34,7 +33,6 @@ private RpcClientAttributesExtractor(RpcAttributesGetter<REQUEST> getter) {
    * This method is internal and is hence not for public use. Its API is unstable and can change at
    * any time.
    */
-  @UnstableApi
   @Override
   public SpanKey internalGetSpanKey() {
     return SpanKey.RPC_CLIENT;

diff --git a/...src/main/java/io/opentelemetry/instrumentation/api/instrumenter/rpc/RpcClientMetrics.java b/...src/main/java/io/opentelemetry/instrumentation/api/instrumenter/rpc/RpcClientMetrics.java
@@ -14,7 +14,6 @@
 import io.opentelemetry.api.metrics.Meter;
 import io.opentelemetry.context.Context;
 import io.opentelemetry.context.ContextKey;
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
 import io.opentelemetry.instrumentation.api.instrumenter.RequestListener;
 import io.opentelemetry.instrumentation.api.instrumenter.RequestMetrics;
 import java.util.concurrent.TimeUnit;
@@ -25,7 +24,6 @@
  * href="https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/semantic_conventions/rpc.md#rpc-client">RPC
  * client metrics</a>.
  */
-@UnstableApi
 public final class RpcClientMetrics implements RequestListener {
 
   private static final double NANOS_PER_MS = TimeUnit.MILLISECONDS.toNanos(1);
@@ -51,7 +49,6 @@ private RpcClientMetrics(Meter meter) {
    * RpcClientMetrics} on an {@link
    * io.opentelemetry.instrumentation.api.instrumenter.InstrumenterBuilder}.
    */
-  @UnstableApi
   public static RequestMetrics get() {
     return RpcClientMetrics::new;
   }

diff --git a/...src/main/java/io/opentelemetry/instrumentation/api/instrumenter/rpc/RpcServerMetrics.java b/...src/main/java/io/opentelemetry/instrumentation/api/instrumenter/rpc/RpcServerMetrics.java
@@ -14,7 +14,6 @@
 import io.opentelemetry.api.metrics.Meter;
 import io.opentelemetry.context.Context;
 import io.opentelemetry.context.ContextKey;
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
 import io.opentelemetry.instrumentation.api.instrumenter.RequestListener;
 import io.opentelemetry.instrumentation.api.instrumenter.RequestMetrics;
 import java.util.concurrent.TimeUnit;
@@ -25,7 +24,6 @@
  * href="https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/semantic_conventions/rpc.md#rpc-server">RPC
  * server metrics</a>.
  */
-@UnstableApi
 public final class RpcServerMetrics implements RequestListener {
 
   private static final double NANOS_PER_MS = TimeUnit.MILLISECONDS.toNanos(1);
@@ -51,7 +49,6 @@ private RpcServerMetrics(Meter meter) {
    * RpcServerMetrics} on an {@link
    * io.opentelemetry.instrumentation.api.instrumenter.InstrumenterBuilder}.
    */
-  @UnstableApi
   public static RequestMetrics get() {
     return RpcServerMetrics::new;
   }

diff --git a/...ation-api/src/main/java/io/opentelemetry/instrumentation/api/annotations/UnstableApi.java b/...ation-api/src/main/java/io/opentelemetry/instrumentation/api/annotations/UnstableApi.java
diff --git a/instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/config/Config.java b/instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/config/Config.java
@@ -9,7 +9,6 @@
 import static java.util.logging.Level.FINE;
 
 import com.google.auto.value.AutoValue;
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
 import java.time.Duration;
 import java.util.List;
 import java.util.Map;
@@ -56,7 +55,6 @@ static Config create(Map<String, String> allProperties) {
    * <p>This method is internal and is hence not for public use. Its API is unstable and can change
    * at any time.
    */
-  @UnstableApi
   public static void internalInitializeConfig(Config config) {
     if (instance != null) {
       logger.warning("Config#INSTANCE was already set earlier");

diff --git a/.../src/main/java/io/opentelemetry/instrumentation/api/instrumenter/InstrumenterBuilder.java b/.../src/main/java/io/opentelemetry/instrumentation/api/instrumenter/InstrumenterBuilder.java
@@ -14,7 +14,6 @@
 import io.opentelemetry.context.Context;
 import io.opentelemetry.context.propagation.TextMapGetter;
 import io.opentelemetry.context.propagation.TextMapSetter;
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
 import io.opentelemetry.instrumentation.api.config.Config;
 import io.opentelemetry.instrumentation.api.internal.SpanKey;
 import io.opentelemetry.instrumentation.api.internal.SpanKeyProvider;
@@ -127,7 +126,6 @@ public InstrumenterBuilder<REQUEST, RESPONSE> addRequestListener(RequestListener
   }
 
   /** Adds a {@link RequestMetrics} whose metrics will be recorded for request start and end. */
-  @UnstableApi
   public InstrumenterBuilder<REQUEST, RESPONSE> addRequestMetrics(RequestMetrics factory) {
     requestListeners.add(factory.create(meter));
     return this;

diff --git a/...n-api/src/main/java/io/opentelemetry/instrumentation/api/instrumenter/RequestMetrics.java b/...n-api/src/main/java/io/opentelemetry/instrumentation/api/instrumenter/RequestMetrics.java
@@ -6,11 +6,9 @@
 package io.opentelemetry.instrumentation.api.instrumenter;
 
 import io.opentelemetry.api.metrics.Meter;
-import io.opentelemetry.instrumentation.api.annotations.UnstableApi;
 
 /** A factory for a {@link RequestListener} for recording metrics using a {@link Meter}. */
 @FunctionalInterface
-@UnstableApi
 public interface RequestMetrics {
   /** Returns a {@link RequestListener} for recording metrics using the given {@link Meter}. */
   RequestListener create(Meter meter);