fix clirr

Author: rahul2393

google-cloud-spanner/clirr-ignored-differences.xml

@@ -349,20 +349,6 @@
     <method>com.google.cloud.spanner.spi.v1.SpannerRpc$StreamingCall executeQuery(com.google.spanner.v1.ExecuteSqlRequest, com.google.cloud.spanner.spi.v1.SpannerRpc$ResultStreamConsumer, java.util.Map)</method>
     <to>com.google.cloud.spanner.spi.v1.SpannerRpc$StreamingCall executeQuery(com.google.spanner.v1.ExecuteSqlRequest, java.util.Map, boolean)</to>
   </difference>
-  <!-- (CompositeTracer is an internal API) -->
-  <difference>
-    <differenceType>7005</differenceType>
-    <className>com/google/cloud/spanner/CompositeTracer$CompositeTracer</className>
-    <method>void recordGFELatency(java.lang.Long)</method>
-    <to>void recordGFELatency(java.lang.Float)</to>
-  </difference>
-  <!-- (CompositeTracer is an internal API) -->
-  <difference>
-    <differenceType>7005</differenceType>
-    <className>com/google/cloud/spanner/CompositeTracer$CompositeTracer</className>
-    <method>void recordAFELatency(java.lang.Long)</method>
-    <to>void recordAFELatency(java.lang.Float)</to>
-  </difference>
   <difference>
     <differenceType>7006</differenceType>
     <className>com/google/cloud/spanner/spi/v1/GapicSpannerRpc</className>

google-cloud-spanner/src/main/java/com/google/cloud/spanner/CompositeTracer.java

@@ -191,10 +191,11 @@ public void addAttributes(Map<String, String> attributes) {
     }
   }
 
-  public void recordGFELatency(Float gfeLatency) {
+  @Deprecated
+  public void recordGFELatency(Long gfeLatency) {
     for (ApiTracer child : children) {
       if (child instanceof BuiltInMetricsTracer) {
-        ((BuiltInMetricsTracer) child).recordGFELatency(gfeLatency);
+        ((BuiltInMetricsTracer) child).recordGFELatency(Float.valueOf(gfeLatency));
       }
     }
   }
@@ -207,10 +208,11 @@ public void recordGfeHeaderMissingCount(Long value) {
     }
   }
 
-  public void recordAFELatency(Float afeLatency) {
+  @Deprecated
+  public void recordAFELatency(Long afeLatency) {
     for (ApiTracer child : children) {
       if (child instanceof BuiltInMetricsTracer) {
-        ((BuiltInMetricsTracer) child).recordAFELatency(afeLatency);
+        ((BuiltInMetricsTracer) child).recordAFELatency(Float.valueOf(afeLatency));
       }
     }
   }
@@ -222,4 +224,20 @@ public void recordAfeHeaderMissingCount(Long value) {
       }
     }
   }
+
+  public void recordGFELatency(Float gfeLatency) {
+    for (ApiTracer child : children) {
+      if (child instanceof BuiltInMetricsTracer) {
+        ((BuiltInMetricsTracer) child).recordGFELatency(gfeLatency);
+      }
+    }
+  }
+
+  public void recordAFELatency(Float afeLatency) {
+    for (ApiTracer child : children) {
+      if (child instanceof BuiltInMetricsTracer) {
+        ((BuiltInMetricsTracer) child).recordAFELatency(afeLatency);
+      }
+    }
+  }
 }