Skip to content

Commit 4f40665

Browse files
committed
early return
1 parent 1bbc240 commit 4f40665

1 file changed

Lines changed: 17 additions & 20 deletions

File tree

java-bigquery-jdbc/src/main/java/com/google/cloud/bigquery/jdbc/BigQueryJdbcOpenTelemetry.java

Lines changed: 17 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -329,27 +329,24 @@ public static OpenTelemetry getOpenTelemetry(
329329
.addPropertiesSupplier(() -> props)
330330
.addSpanExporterCustomizer(
331331
(spanExporter, configProperties) -> {
332-
if (gcpTelemetryCredentials != null) {
333-
try {
334-
Credentials credentials =
335-
resolveCredentialsFromString(gcpTelemetryCredentials);
336-
if (spanExporter instanceof OtlpHttpSpanExporter) {
337-
return ((OtlpHttpSpanExporter) spanExporter)
338-
.toBuilder()
339-
.setHeaders(() -> getAuthHeaders(credentials))
340-
.build();
341-
}
342-
if (spanExporter instanceof OtlpGrpcSpanExporter) {
343-
return ((OtlpGrpcSpanExporter) spanExporter)
344-
.toBuilder()
345-
.setHeaders(() -> getAuthHeaders(credentials))
346-
.build();
347-
}
348-
} catch (Exception e) {
349-
LOG.warning(
350-
e,
351-
"Failed to resolve telemetry credentials. Telemetry will be exported using default OpenTelemetry configuration (custom authentication headers will not be injected).");
332+
if (gcpTelemetryCredentials == null) {
333+
return spanExporter;
334+
}
335+
try {
336+
Credentials credentials =
337+
resolveCredentialsFromString(gcpTelemetryCredentials);
338+
if (spanExporter instanceof OtlpHttpSpanExporter) {
339+
return ((OtlpHttpSpanExporter) spanExporter)
340+
.toBuilder().setHeaders(() -> getAuthHeaders(credentials)).build();
341+
}
342+
if (spanExporter instanceof OtlpGrpcSpanExporter) {
343+
return ((OtlpGrpcSpanExporter) spanExporter)
344+
.toBuilder().setHeaders(() -> getAuthHeaders(credentials)).build();
352345
}
346+
} catch (Exception e) {
347+
LOG.warning(
348+
e,
349+
"Failed to resolve telemetry credentials. Telemetry will be exported using default OpenTelemetry configuration (custom authentication headers will not be injected).");
353350
}
354351
return spanExporter;
355352
})

0 commit comments

Comments
 (0)