Use separate id generator; rename attribute name.

HailongWen · mattwhisenhunt · commit 86bc0ae11e28 · 2018-02-06T12:29:40.000-08:00
diff --git a/google-http-client/src/main/java/com/google/api/client/http/HttpRequest.java b/google-http-client/src/main/java/com/google/api/client/http/HttpRequest.java
@@ -869,14 +869,15 @@ public HttpResponse execute() throws IOException {
         .spanBuilder(OpenCensusUtils.SPAN_NAME_HTTP_REQUEST_EXECUTE)
         .setRecordEvents(OpenCensusUtils.isRecordEvent())
         .startSpan();
-    long idGenerator = 0L;
+    long sentIdGenerator = 0L;
+    long recvIdGenerator = 0L;
 
     do {
       span.addAnnotation(
           Annotation.fromDescriptionAndAttributes(
               "retry",
               Collections.<String, AttributeValue>singletonMap(
-                  "number of retry",
+                  "number",
                   AttributeValue.longAttributeValue(numRetries - retriesRemaining))));
       // Cleanup any unneeded response from a previous iteration
       if (response != null) {
@@ -1005,12 +1006,12 @@ public HttpResponse execute() throws IOException {
       // switch tracing scope to current span
       Scope ws = tracer.withSpan(span);
       OpenCensusUtils.recordSentMessageEvent(
-          span, idGenerator++, lowLevelHttpRequest.getContentLength());
+          span, sentIdGenerator++, lowLevelHttpRequest.getContentLength());
       try {
         LowLevelHttpResponse lowLevelHttpResponse = lowLevelHttpRequest.execute();
         if (lowLevelHttpResponse != null) {
           OpenCensusUtils.recordReceivedMessageEvent(
-              span, idGenerator++, lowLevelHttpResponse.getContentLength());
+              span, recvIdGenerator++, lowLevelHttpResponse.getContentLength());
         }
         // Flag used to indicate if an exception is thrown before the response is constructed.
         boolean responseConstructed = false;
