fix: move client level context init. closer to where its first used

diegomarquezp · diegomarquezp · commit 56c045589004 · 2026-03-30T10:24:57.000-04:00
diff --git a/sdk-platform-java/gax-java/gax/src/main/java/com/google/api/gax/tracing/GoldenSignalsMetricsTracerFactory.java b/sdk-platform-java/gax-java/gax/src/main/java/com/google/api/gax/tracing/GoldenSignalsMetricsTracerFactory.java
@@ -79,11 +79,11 @@ public ApiTracerFactory withContext(ApiTracerContext context) {
     if (context == null) {
       return new BaseApiTracerFactory();
     }
-    this.clientLevelTracerContext = context;
     LibraryMetadata metadata = context.libraryMetadata();
     if (metadata == null || metadata.isEmpty() || Strings.isNullOrEmpty(metadata.artifactName())) {
       return new BaseApiTracerFactory();
     }
+    this.clientLevelTracerContext = context;
     this.metricsRecorder =
         new GoldenSignalsMetricsRecorder(
             openTelemetry, clientLevelTracerContext.libraryMetadata().artifactName());

Original file line number	Diff line number	Diff line change
`@@ -79,11 +79,11 @@ public ApiTracerFactory withContext(ApiTracerContext context) {`
`79`	`79`	`if (context == null) {`
`80`	`80`	`return new BaseApiTracerFactory();`
`81`	`81`	`}`
`82`		`- this.clientLevelTracerContext = context;`
`83`	`82`	`LibraryMetadata metadata = context.libraryMetadata();`
`84`	`83`	`if (metadata == null \|\| metadata.isEmpty() \|\| Strings.isNullOrEmpty(metadata.artifactName())) {`
`85`	`84`	`return new BaseApiTracerFactory();`
`86`	`85`	`}`
	`86`	`+ this.clientLevelTracerContext = context;`
`87`	`87`	`this.metricsRecorder =`
`88`	`88`	`new GoldenSignalsMetricsRecorder(`
`89`	`89`	`openTelemetry, clientLevelTracerContext.libraryMetadata().artifactName());`