fix: remove duplicate logic

diegomarquezp · diegomarquezp · commit a4a85b32f85e · 2026-03-16T13:38:16.000-04:00
diff --git a/gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java b/gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java
@@ -179,13 +179,8 @@ public Map<String, Object> getAttemptAttributes() {
             ObservabilityAttributes.ARTIFACT_ATTRIBUTE, libraryMetadata().artifactName());
       }
     }
-    if (transport() == Transport.GRPC) {
-      if (!Strings.isNullOrEmpty(fullMethodName())) {
-        attributes.put(ObservabilityAttributes.GRPC_RPC_METHOD_ATTRIBUTE, fullMethodName());
-      }
-      if (Strings.isNullOrEmpty(fullMethodName())) {
-        attributes.put(ObservabilityAttributes.GRPC_RPC_METHOD_ATTRIBUTE, fullMethodName());
-      }
+    if (transport() == Transport.GRPC && !Strings.isNullOrEmpty(fullMethodName())) {
+      attributes.put(ObservabilityAttributes.GRPC_RPC_METHOD_ATTRIBUTE, fullMethodName());
     }
     if (transport() == Transport.HTTP && !Strings.isNullOrEmpty(httpMethod())) {
       attributes.put(ObservabilityAttributes.HTTP_METHOD_ATTRIBUTE, httpMethod());

Original file line number	Diff line number	Diff line change
`@@ -179,13 +179,8 @@ public Map<String, Object> getAttemptAttributes() {`
`179`	`179`	`ObservabilityAttributes.ARTIFACT_ATTRIBUTE, libraryMetadata().artifactName());`
`180`	`180`	`}`
`181`	`181`	`}`
`182`		`- if (transport() == Transport.GRPC) {`
`183`		`- if (!Strings.isNullOrEmpty(fullMethodName())) {`
`184`		`- attributes.put(ObservabilityAttributes.GRPC_RPC_METHOD_ATTRIBUTE, fullMethodName());`
`185`		`- }`
`186`		`- if (Strings.isNullOrEmpty(fullMethodName())) {`
`187`		`- attributes.put(ObservabilityAttributes.GRPC_RPC_METHOD_ATTRIBUTE, fullMethodName());`
`188`		`- }`
	`182`	`+ if (transport() == Transport.GRPC && !Strings.isNullOrEmpty(fullMethodName())) {`
	`183`	`+ attributes.put(ObservabilityAttributes.GRPC_RPC_METHOD_ATTRIBUTE, fullMethodName());`
`189`	`184`	`}`
`190`	`185`	`if (transport() == Transport.HTTP && !Strings.isNullOrEmpty(httpMethod())) {`
`191`	`186`	`attributes.put(ObservabilityAttributes.HTTP_METHOD_ATTRIBUTE, httpMethod());`