Skip to content

Commit

Permalink
update after #6546 merged
Browse files Browse the repository at this point in the history
  • Loading branch information
chungngoops committed Jul 24, 2024
1 parent 8e62c93 commit 300712b
Showing 1 changed file with 13 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@

package io.opentelemetry.exporter.zipkin;

import io.opentelemetry.api.internal.InstrumentationUtil;
import io.opentelemetry.api.metrics.MeterProvider;
import io.opentelemetry.exporter.internal.ExporterMetrics;
import io.opentelemetry.exporter.internal.InstrumentationUtil;
import io.opentelemetry.sdk.common.CompletableResultCode;
import io.opentelemetry.sdk.internal.ThrottlingLogger;
import io.opentelemetry.sdk.trace.data.SpanData;
Expand Down Expand Up @@ -77,17 +77,18 @@ public CompletableResultCode export(Collection<SpanData> spanDataList) {
}

CompletableResultCode resultCode = new CompletableResultCode();
InstrumentationUtil.suppressInstrumentation(() -> {
try {
sender.send(encodedSpans);
exporterMetrics.addSuccess(numItems);
resultCode.succeed();
} catch (IOException | RuntimeException e) {
exporterMetrics.addFailed(numItems);
logger.log(Level.WARNING, "Failed to export spans", e);
resultCode.fail();
}
});
InstrumentationUtil.suppressInstrumentation(
() -> {
try {
sender.send(encodedSpans);
exporterMetrics.addSuccess(numItems);
resultCode.succeed();
} catch (IOException | RuntimeException e) {
exporterMetrics.addFailed(numItems);
logger.log(Level.WARNING, "Failed to export spans", e);
resultCode.fail();
}
});
return resultCode;
}

Expand Down

0 comments on commit 300712b

Please sign in to comment.