update the jaeger-client dependency to the latest (#3172)

open-telemetry · Apr 22, 2021 · ccb03c8 · ccb03c8
1 parent 7e2c34a
commit ccb03c8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/dependencyManagement/build.gradle.kts b/dependencyManagement/build.gradle.kts
@@ -83,7 +83,7 @@ val DEPENDENCIES = listOf(
         "edu.berkeley.cs.jqf:jqf-fuzz:1.6",
         "eu.rekawek.toxiproxy:toxiproxy-java:2.1.4",
         "io.github.netmikey.logunit:logunit-jul:1.1.0",
-        "io.jaegertracing:jaeger-client:1.5.0",
+        "io.jaegertracing:jaeger-client:1.6.0",
         "io.opentracing:opentracing-api:0.33.0",
         "io.zipkin.zipkin2:zipkin-junit:2.23.2",
         "junit:junit:4.13.2",

diff --git a/...rc/main/java/io/opentelemetry/exporter/jaeger/thrift/JaegerThriftSpanExporterBuilder.java b/...rc/main/java/io/opentelemetry/exporter/jaeger/thrift/JaegerThriftSpanExporterBuilder.java
@@ -7,6 +7,7 @@
 
 import io.jaegertracing.thrift.internal.senders.HttpSender;
 import io.jaegertracing.thrift.internal.senders.ThriftSender;
+import org.apache.thrift.transport.TTransportException;
 
 /** Builder utility for this exporter. */
 public final class JaegerThriftSpanExporterBuilder {
@@ -46,7 +47,11 @@ public JaegerThriftSpanExporterBuilder setEndpoint(String endpoint) {
    */
   public JaegerThriftSpanExporter build() {
     if (thriftSender == null) {
-      thriftSender = new HttpSender.Builder(endpoint).build();
+      try {
+        thriftSender = new HttpSender.Builder(endpoint).build();
+      } catch (TTransportException e) {
+        throw new IllegalStateException("Failed to construct a thrift HttpSender.", e);
+      }
     }
     return new JaegerThriftSpanExporter(thriftSender);
   }