Deprecate jaeger exporters (#5190)

README.md

@@ -253,13 +253,20 @@ dependency as follows, replacing `{{artifact-id}}` with the value from the "Arti
 |-----------------------------------------------------|-------------------------------------------------------------------------------------|---------------------------------------|-------------------------------------------------------------|-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
 | [OTLP Exporters](./exporters/otlp/all)              | OTLP gRPC & HTTP exporters, including traces, metrics, and logs                     | `opentelemetry-exporter-otlp`         | <!--VERSION_STABLE-->1.27.0<!--/VERSION_STABLE-->           | [![Javadocs](https://www.javadoc.io/badge/io.opentelemetry/opentelemetry-exporter-otlp.svg)](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-exporter-otlp)                   |
 | [OTLP Common](./exporters/otlp/common)              | Shared OTLP components (internal)                                                   | `opentelemetry-exporter-otlp-common`  | <!--VERSION_STABLE-->1.27.0<!--/VERSION_STABLE-->           | [![Javadocs](https://www.javadoc.io/badge/io.opentelemetry/opentelemetry-exporter-otlp-common.svg)](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-exporter-otlp-common)     |
-| [Jaeger gRPC Exporter](./exporters/jaeger)          | Jaeger gRPC trace exporter                                                          | `opentelemetry-exporter-jaeger`       | <!--VERSION_STABLE-->1.27.0<!--/VERSION_STABLE-->           | [![Javadocs](https://www.javadoc.io/badge/io.opentelemetry/opentelemetry-exporter-jaeger.svg)](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-exporter-jaeger)               |
-| [Jaeger Thrift Exporter](./exporters/jaeger-thrift) | Jaeger thrift trace exporter                                                        | `opentelemetry-exporter-jaeger-thift` | <!--VERSION_STABLE-->1.27.0<!--/VERSION_STABLE-->           | [![Javadocs](https://www.javadoc.io/badge/io.opentelemetry/opentelemetry-exporter-jaeger-thrift.svg)](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-exporter-jaeger-thrift) |
+| [Jaeger gRPC Exporter](./exporters/jaeger)          | Jaeger gRPC trace exporter (deprecated [1])                                                            | `opentelemetry-exporter-jaeger`       | <!--VERSION_STABLE-->1.27.0<!--/VERSION_STABLE-->           | [![Javadocs](https://www.javadoc.io/badge/io.opentelemetry/opentelemetry-exporter-jaeger.svg)](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-exporter-jaeger)               |
+| [Jaeger Thrift Exporter](./exporters/jaeger-thrift) | Jaeger thrift trace exporter (deprecated [1])                                                        | `opentelemetry-exporter-jaeger-thift` | <!--VERSION_STABLE-->1.27.0<!--/VERSION_STABLE-->           | [![Javadocs](https://www.javadoc.io/badge/io.opentelemetry/opentelemetry-exporter-jaeger-thrift.svg)](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-exporter-jaeger-thrift) |
 | [Logging Exporter](./exporters/logging)             | Logging exporters, including metrics, traces, and logs                              | `opentelemetry-exporter-logging`      | <!--VERSION_STABLE-->1.27.0<!--/VERSION_STABLE-->           | [![Javadocs](https://www.javadoc.io/badge/io.opentelemetry/opentelemetry-exporter-logging.svg)](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-exporter-logging)             |
 | [Zipkin Exporter](./exporters/zipkin)               | Zipkin trace exporter                                                               | `opentelemetry-exporter-zipkin`       | <!--VERSION_STABLE-->1.27.0<!--/VERSION_STABLE-->           | [![Javadocs](https://www.javadoc.io/badge/io.opentelemetry/opentelemetry-exporter-zipkin.svg)](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-exporter-zipkin)               |
 | [Prometheus Exporter](./exporters/prometheus)       | Prometheus metric exporter                                                          | `opentelemetry-exporter-prometheus`   | <!--VERSION_UNSTABLE-->1.27.0-alpha<!--/VERSION_UNSTABLE--> | [![Javadocs](https://www.javadoc.io/badge/io.opentelemetry/opentelemetry-exporter-prometheus.svg)](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-exporter-prometheus)       |
 | [Exporter Common](./exporters/common)               | Shared exporter components (internal)                                               | `opentelemetry-exporter-common`       | <!--VERSION_STABLE-->1.27.0<!--/VERSION_STABLE-->           | [![Javadocs](https://www.javadoc.io/badge/io.opentelemetry/opentelemetry-exporter-common.svg)](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-exporter-common)               |
 
+**[1]**: Jaeger now
+has [native support for OTLP](https://opentelemetry.io/blog/2022/jaeger-native-otlp/) and jaeger
+exporters are now deprecated. `opentelemetry-exporter-jaeger-thrift` will continue to be published
+until 1.34.0 (January 2024) but no new PRs will be accepted except security related bugfixes. After
+1.34.0, `io.opentelemetry:opentelemetry-bom` will reference the last published version, but no
+additional versions will be published.
+
 ### SDK Extensions
 
 | Component                                                                     | Description                                                                                    | Artifact ID                                         | Version                                                     | Javadoc                                                                                                                                                                                                           |

docs/apidiffs/current_vs_latest/opentelemetry-exporter-jaeger-thrift.txt

@@ -1,2 +1,7 @@
 Comparing source compatibility of  against 
-No changes.
+===  UNCHANGED CLASS: PUBLIC FINAL io.opentelemetry.exporter.jaeger.thrift.JaegerThriftSpanExporter  (not serializable)
+	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
+	+++  NEW ANNOTATION: java.lang.Deprecated
+===  UNCHANGED CLASS: PUBLIC FINAL io.opentelemetry.exporter.jaeger.thrift.JaegerThriftSpanExporterBuilder  (not serializable)
+	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
+	+++  NEW ANNOTATION: java.lang.Deprecated

docs/apidiffs/current_vs_latest/opentelemetry-exporter-jaeger.txt

@@ -1,2 +1,7 @@
 Comparing source compatibility of  against 
-No changes.
+===  UNCHANGED CLASS: PUBLIC FINAL io.opentelemetry.exporter.jaeger.JaegerGrpcSpanExporter  (not serializable)
+	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
+	+++  NEW ANNOTATION: java.lang.Deprecated
+===  UNCHANGED CLASS: PUBLIC FINAL io.opentelemetry.exporter.jaeger.JaegerGrpcSpanExporterBuilder  (not serializable)
+	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
+	+++  NEW ANNOTATION: java.lang.Deprecated

exporters/jaeger-thrift/src/main/java/io/opentelemetry/exporter/jaeger/thrift/JaegerThriftSpanExporter.java

@@ -29,8 +29,15 @@
 import java.util.stream.Collectors;
 import javax.annotation.concurrent.ThreadSafe;
 
-/** Exports spans to Jaeger via Thrift, using Jaeger's thrift model. */
+/**
+ * Exports spans to Jaeger via Thrift, using Jaeger's thrift model.
+ *
+ * @deprecated Use {@code OtlpGrpcSpanExporter} or {@code OtlpHttpSpanExporter} from <a
+ *     href="https://github.com/open-telemetry/opentelemetry-java/tree/main/exporters/otlp/all">opentelemetry-exporter-otlp</a>
+ *     instead.
+ */
 @ThreadSafe
+@Deprecated
 public final class JaegerThriftSpanExporter implements SpanExporter {
 
   static final String DEFAULT_ENDPOINT = "http://localhost:14268/api/traces";

exporters/jaeger-thrift/src/main/java/io/opentelemetry/exporter/jaeger/thrift/JaegerThriftSpanExporterBuilder.java

@@ -10,7 +10,14 @@
 import javax.annotation.Nullable;
 import org.apache.thrift.transport.TTransportException;
 
-/** Builder utility for this exporter. */
+/**
+ * Builder utility for this exporter.
+ *
+ * @deprecated Use {@code OtlpGrpcSpanExporter} or {@code OtlpHttpSpanExporter} from <a
+ *     href="https://github.com/open-telemetry/opentelemetry-java/tree/main/exporters/otlp/all">opentelemetry-exporter-otlp</a>
+ *     instead.
+ */
+@Deprecated
 public final class JaegerThriftSpanExporterBuilder {
 
   private String endpoint = JaegerThriftSpanExporter.DEFAULT_ENDPOINT;

exporters/jaeger-thrift/src/test/java/io/opentelemetry/exporter/jaeger/thrift/JaegerThriftIntegrationTest.java

@@ -33,6 +33,7 @@
 import org.testcontainers.junit.jupiter.Testcontainers;
 
 @Testcontainers(disabledWithoutDocker = true)
+@SuppressWarnings("deprecation") // Testing deprecated code
 class JaegerThriftIntegrationTest {
   private static final OkHttpClient client = new OkHttpClient();
 

exporters/jaeger-thrift/src/test/java/io/opentelemetry/exporter/jaeger/thrift/JaegerThriftSpanExporterTest.java

@@ -46,6 +46,7 @@
 import org.mockito.junit.jupiter.MockitoExtension;
 
 @ExtendWith(MockitoExtension.class)
+@SuppressWarnings("deprecation") // Testing deprecated code
 class JaegerThriftSpanExporterTest {
 
   private static final String TRACE_ID = "a0000000000000000000000000abc123";

exporters/jaeger/src/main/java/io/opentelemetry/exporter/jaeger/JaegerGrpcSpanExporter.java

@@ -19,8 +19,15 @@
 import java.util.stream.Collectors;
 import javax.annotation.concurrent.ThreadSafe;
 
-/** Exports spans to Jaeger via gRPC, using Jaeger's protobuf model. */
+/**
+ * Exports spans to Jaeger via gRPC, using Jaeger's protobuf model.
+ *
+ * @deprecated Use {@code OtlpGrpcSpanExporter} or {@code OtlpHttpSpanExporter} from <a
+ *     href="https://github.com/open-telemetry/opentelemetry-java/tree/main/exporters/otlp/all">opentelemetry-exporter-otlp</a>
+ *     instead.
+ */
 @ThreadSafe
+@Deprecated
 public final class JaegerGrpcSpanExporter implements SpanExporter {
 
   private static final String DEFAULT_HOST_NAME = "unknown";

exporters/jaeger/src/main/java/io/opentelemetry/exporter/jaeger/JaegerGrpcSpanExporterBuilder.java

@@ -19,7 +19,14 @@
 import javax.net.ssl.SSLContext;
 import javax.net.ssl.X509TrustManager;
 
-/** Builder utility for this exporter. */
+/**
+ * Builder utility for this exporter.
+ *
+ * @deprecated Use {@code OtlpGrpcSpanExporter} or {@code OtlpHttpSpanExporter} from <a
+ *     href="https://github.com/open-telemetry/opentelemetry-java/tree/main/exporters/otlp/all">opentelemetry-exporter-otlp</a>
+ *     instead.
+ */
+@Deprecated
 public final class JaegerGrpcSpanExporterBuilder {
 
   private static final String GRPC_SERVICE_NAME = "jaeger.api_v2.CollectorService";

exporters/jaeger/src/main/java/io/opentelemetry/exporter/jaeger/internal/JaegerGrpcSpanExporterProvider.java

@@ -5,19 +5,23 @@
 
 package io.opentelemetry.exporter.jaeger.internal;
 
-import io.opentelemetry.exporter.jaeger.JaegerGrpcSpanExporter;
-import io.opentelemetry.exporter.jaeger.JaegerGrpcSpanExporterBuilder;
 import io.opentelemetry.sdk.autoconfigure.spi.ConfigProperties;
 import io.opentelemetry.sdk.autoconfigure.spi.traces.ConfigurableSpanExporterProvider;
 import io.opentelemetry.sdk.trace.export.SpanExporter;
 import java.time.Duration;
 
 /**
- * {@link SpanExporter} SPI implementation for {@link JaegerGrpcSpanExporter}.
+ * {@link SpanExporter} SPI implementation for {@link
+ * io.opentelemetry.exporter.jaeger.JaegerGrpcSpanExporter}.
  *
  * <p>This class is internal and is hence not for public use. Its APIs are unstable and can change
  * at any time.
+ *
+ * @deprecated Use {@code OtlpGrpcSpanExporter} or {@code OtlpHttpSpanExporter} from <a
+ *     href="https://github.com/open-telemetry/opentelemetry-java/tree/main/exporters/otlp/all">opentelemetry-exporter-otlp</a>
+ *     instead.
  */
+@Deprecated
 public class JaegerGrpcSpanExporterProvider implements ConfigurableSpanExporterProvider {
   @Override
   public String getName() {
@@ -26,7 +30,8 @@ public String getName() {
 
   @Override
   public SpanExporter createExporter(ConfigProperties config) {
-    JaegerGrpcSpanExporterBuilder builder = JaegerGrpcSpanExporter.builder();
+    io.opentelemetry.exporter.jaeger.JaegerGrpcSpanExporterBuilder builder =
+        io.opentelemetry.exporter.jaeger.JaegerGrpcSpanExporter.builder();
 
     String endpoint = config.getString("otel.exporter.jaeger.endpoint");
     if (endpoint != null) {

exporters/jaeger/src/test/java/io/opentelemetry/exporter/jaeger/JaegerGrpcSpanExporterTest.java

@@ -62,6 +62,7 @@
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 
+@SuppressWarnings("deprecation") // Testing deprecated code
 class JaegerGrpcSpanExporterTest {
   private static final BlockingQueue<Collector.PostSpansRequest> postedRequests =
       new LinkedBlockingDeque<>();

exporters/jaeger/src/test/java/io/opentelemetry/exporter/jaeger/JaegerIntegrationTest.java

@@ -29,6 +29,7 @@
 import org.testcontainers.junit.jupiter.Testcontainers;
 
 @Testcontainers(disabledWithoutDocker = true)
+@SuppressWarnings("deprecation") // Testing deprecated code
 class JaegerIntegrationTest {
   private static final OkHttpClient client = new OkHttpClient();
 

exporters/jaeger/src/test/java/io/opentelemetry/exporter/jaeger/internal/JaegerGrpcSpanExporterProviderTest.java

@@ -7,7 +7,6 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-import io.opentelemetry.exporter.jaeger.JaegerGrpcSpanExporter;
 import io.opentelemetry.sdk.autoconfigure.spi.internal.DefaultConfigProperties;
 import io.opentelemetry.sdk.trace.export.SpanExporter;
 import java.util.Collections;
@@ -16,6 +15,7 @@
 import okhttp3.HttpUrl;
 import org.junit.jupiter.api.Test;
 
+@SuppressWarnings("deprecation") // Testing deprecated code
 class JaegerGrpcSpanExporterProviderTest {
 
   private static final JaegerGrpcSpanExporterProvider provider =
@@ -30,7 +30,8 @@ void getName() {
   void createExporter_Default() {
     try (SpanExporter spanExporter =
         provider.createExporter(DefaultConfigProperties.createForTest(Collections.emptyMap()))) {
-      assertThat(spanExporter).isInstanceOf(JaegerGrpcSpanExporter.class);
+      assertThat(spanExporter)
+          .isInstanceOf(io.opentelemetry.exporter.jaeger.JaegerGrpcSpanExporter.class);
       assertThat(spanExporter)
           .extracting("delegate")
           .extracting("client")
@@ -52,7 +53,8 @@ void createExporter_WithConfiguration() {
 
     try (SpanExporter spanExporter =
         provider.createExporter(DefaultConfigProperties.createForTest(config))) {
-      assertThat(spanExporter).isInstanceOf(JaegerGrpcSpanExporter.class);
+      assertThat(spanExporter)
+          .isInstanceOf(io.opentelemetry.exporter.jaeger.JaegerGrpcSpanExporter.class);
       assertThat(spanExporter)
           .extracting("delegate")
           .extracting("client")

sdk-extensions/autoconfigure/src/testFullConfig/java/io/opentelemetry/sdk/autoconfigure/SpanExporterConfigurationTest.java

@@ -9,7 +9,6 @@
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
 import com.google.common.collect.ImmutableMap;
-import io.opentelemetry.exporter.jaeger.JaegerGrpcSpanExporter;
 import io.opentelemetry.exporter.logging.LoggingSpanExporter;
 import io.opentelemetry.exporter.logging.otlp.OtlpJsonLoggingSpanExporter;
 import io.opentelemetry.exporter.otlp.trace.OtlpGrpcSpanExporter;
@@ -24,14 +23,15 @@
 class SpanExporterConfigurationTest {
 
   @Test
+  @SuppressWarnings("deprecation") // Testing deprecated jaeger exporter
   void configureExporter_KnownSpiExportersOnClasspath() {
     NamedSpiManager<SpanExporter> spiExportersManager =
         SpanExporterConfiguration.spanExporterSpiManager(
             DefaultConfigProperties.createForTest(Collections.emptyMap()),
             SpanExporterConfigurationTest.class.getClassLoader());
 
     assertThat(SpanExporterConfiguration.configureExporter("jaeger", spiExportersManager))
-        .isInstanceOf(JaegerGrpcSpanExporter.class);
+        .isInstanceOf(io.opentelemetry.exporter.jaeger.JaegerGrpcSpanExporter.class);
     assertThat(SpanExporterConfiguration.configureExporter("logging", spiExportersManager))
         .isInstanceOf(LoggingSpanExporter.class);
     assertThat(SpanExporterConfiguration.configureExporter("logging-otlp", spiExportersManager))

sdk/trace/src/jmh/java/io/opentelemetry/sdk/trace/ExporterBenchmark.java

@@ -7,7 +7,6 @@
 
 import io.opentelemetry.api.trace.Span;
 import io.opentelemetry.api.trace.Tracer;
-import io.opentelemetry.exporter.jaeger.thrift.JaegerThriftSpanExporter;
 import io.opentelemetry.exporter.otlp.trace.OtlpGrpcSpanExporter;
 import io.opentelemetry.sdk.trace.export.SimpleSpanProcessor;
 import io.opentelemetry.sdk.trace.export.SpanExporter;
@@ -94,12 +93,14 @@ protected OtlpGrpcSpanExporter createExporter(GenericContainer<?> collector) {
     }
   }
 
+  @SuppressWarnings("deprecation") // Benchmarking deprecated code
   public static class JaegerBenchmark extends AbstractProcessorBenchmark {
     @Override
-    protected JaegerThriftSpanExporter createExporter(GenericContainer<?> collector) {
+    protected io.opentelemetry.exporter.jaeger.thrift.JaegerThriftSpanExporter createExporter(
+        GenericContainer<?> collector) {
       String host = collector.getHost();
       int port = collector.getMappedPort(JAEGER_PORT);
-      return JaegerThriftSpanExporter.builder()
+      return io.opentelemetry.exporter.jaeger.thrift.JaegerThriftSpanExporter.builder()
           .setEndpoint("http://" + host + ":" + port + "/api/traces")
           .build();
     }