fix: disable afe_connectivity_error_count metric (#4041)

surbhigarg92 · cloud-java-bot · web-flow · commit f89c1c0517ba · 2025-09-11T22:45:30.000+05:30
* fix: disable afe_connectivity_error_count metric

* chore: generate libraries at Thu Sep 11 14:04:02 UTC 2025

---------

Co-authored-by: cloud-java-bot &lt;cloud-java-bot@google.com&gt;
diff --git a/README.md b/README.md
@@ -41,7 +41,7 @@ If you are using Maven without the BOM, add this to your dependencies:
 <dependency>
   <groupId>com.google.cloud</groupId>
   <artifactId>google-cloud-spanner</artifactId>
-  <version>6.93.0</version>
+  <version>6.99.0</version>
 </dependency>
 
 ```
diff --git a/google-cloud-spanner/src/main/java/com/google/cloud/spanner/spi/v1/HeaderInterceptor.java b/google-cloud-spanner/src/main/java/com/google/cloud/spanner/spi/v1/HeaderInterceptor.java
@@ -189,7 +189,9 @@ private void processHeader(
           float afeLatency = serverTimingMetrics.get(AFE_TIMING_HEADER);
           compositeTracer.recordAFELatency(afeLatency);
         } else {
-          compositeTracer.recordAfeHeaderMissingCount(1L);
+          // Disable afe_connectivity_error_count metric as AFE header is disabled in backend
+          // currently.
+          // compositeTracer.recordAfeHeaderMissingCount(1L);
         }
       }
     } catch (NumberFormatException e) {
diff --git a/google-cloud-spanner/src/test/java/com/google/cloud/spanner/OpenTelemetryBuiltInMetricsTracerTest.java b/google-cloud-spanner/src/test/java/com/google/cloud/spanner/OpenTelemetryBuiltInMetricsTracerTest.java
@@ -390,9 +390,9 @@ public void testNoServerTimingHeader() throws IOException, InterruptedException
 
     assertFalse(checkIfMetricExists(metricReader, BuiltInMetricsConstant.AFE_LATENCIES_NAME));
     assertFalse(checkIfMetricExists(metricReader, BuiltInMetricsConstant.GFE_LATENCIES_NAME));
-    MetricData afeConnectivityMetricData =
-        getMetricData(metricReader, BuiltInMetricsConstant.AFE_CONNECTIVITY_ERROR_NAME);
-    assertThat(getAggregatedValue(afeConnectivityMetricData, expectedAttributes)).isEqualTo(1);
+    // Metric is disabled currently
+    assertFalse(
+        checkIfMetricExists(metricReader, BuiltInMetricsConstant.AFE_CONNECTIVITY_ERROR_NAME));
 
     spannerNoHeader.close();
     serverNoHeader.shutdown();

Original file line number	Diff line number	Diff line change
`@@ -189,7 +189,9 @@ private void processHeader(`
`189`	`189`	`float afeLatency = serverTimingMetrics.get(AFE_TIMING_HEADER);`
`190`	`190`	`compositeTracer.recordAFELatency(afeLatency);`
`191`	`191`	`} else {`
`192`		`- compositeTracer.recordAfeHeaderMissingCount(1L);`
	`192`	`+ // Disable afe_connectivity_error_count metric as AFE header is disabled in backend`
	`193`	`+ // currently.`
	`194`	`+ // compositeTracer.recordAfeHeaderMissingCount(1L);`
`193`	`195`	`}`
`194`	`196`	`}`
`195`	`197`	`} catch (NumberFormatException e) {`