Skip to content

Commit de4bd3b

Browse files
authored
Remove unused code from jaeger remote sampler (#7970)
1 parent 82ecfce commit de4bd3b

File tree

1 file changed

+0
-13
lines changed
  • sdk-extensions/jaeger-remote-sampler/src/main/java/io/opentelemetry/sdk/extension/trace/jaeger/sampler

1 file changed

+0
-13
lines changed

sdk-extensions/jaeger-remote-sampler/src/main/java/io/opentelemetry/sdk/extension/trace/jaeger/sampler/OkHttpGrpcService.java

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
package io.opentelemetry.sdk.extension.trace.jaeger.sampler;
77

8-
import io.opentelemetry.exporter.internal.RetryUtil;
98
import io.opentelemetry.exporter.internal.grpc.GrpcExporterUtil;
109
import io.opentelemetry.exporter.sender.okhttp.internal.GrpcRequestBody;
1110
import io.opentelemetry.sdk.common.CompletableResultCode;
@@ -171,18 +170,6 @@ public CompletableResultCode shutdown() {
171170
return CompletableResultCode.ofSuccess();
172171
}
173172

174-
static boolean isRetryable(Response response) {
175-
// Only retry on gRPC codes which will always come with an HTTP success
176-
if (!response.isSuccessful()) {
177-
return false;
178-
}
179-
180-
// We don't check trailers for retry since retryable error codes always come with response
181-
// headers, not trailers, in practice.
182-
String grpcStatus = response.header(GRPC_STATUS);
183-
return RetryUtil.retryableGrpcStatusCodes().contains(grpcStatus);
184-
}
185-
186173
// From grpc-java
187174

188175
/** Unescape the provided ascii to a unicode {@link String}. */

0 commit comments

Comments
 (0)