feat: remove opencensus (googleapis#834)

* feat: remove beta OpenCensus integration

* remove extra references

* jsr305 dependency no longer needed after removing opencensus

Author: chingor13

google-http-client-findbugs/pom.xml

@@ -70,6 +70,10 @@
           <artifactId>annotations</artifactId>
           <groupId>com.google.code.findbugs</groupId>
         </exclusion>
+        <exclusion>
+          <artifactId>jsr305</artifactId>
+          <groupId>com.google.code.findbugs</groupId>
+        </exclusion>
       </exclusions>
     </dependency>
     <dependency>

google-http-client/pom.xml

@@ -18,14 +18,6 @@
   <build>
     <pluginManagement>
       <plugins>
-        <plugin>
-          <groupId>org.apache.maven.plugins</groupId>
-          <artifactId>maven-dependency-plugin</artifactId>
-          <version>3.1.2</version>
-          <configuration>
-            <ignoredUnusedDeclaredDependencies>io.opencensus:opencensus-impl</ignoredUnusedDeclaredDependencies>
-          </configuration>
-        </plugin>
         <plugin>
           <groupId>org.codehaus.mojo</groupId>
           <artifactId>animal-sniffer-maven-plugin</artifactId>
@@ -131,10 +123,6 @@
       <groupId>org.apache.httpcomponents</groupId>
       <artifactId>httpcore</artifactId>
     </dependency>
-    <dependency>
-      <groupId>com.google.code.findbugs</groupId>
-      <artifactId>jsr305</artifactId>
-    </dependency>
     <dependency>
       <groupId>com.google.guava</groupId>
       <artifactId>guava</artifactId>
@@ -143,14 +131,6 @@
       <groupId>com.google.j2objc</groupId>
       <artifactId>j2objc-annotations</artifactId>
     </dependency>
-    <dependency>
-      <groupId>io.opencensus</groupId>
-      <artifactId>opencensus-api</artifactId>
-    </dependency>
-    <dependency>
-      <groupId>io.opencensus</groupId>
-      <artifactId>opencensus-contrib-http-util</artifactId>
-    </dependency>
 
     <dependency>
       <groupId>com.google.guava</groupId>
@@ -167,15 +147,5 @@
       <artifactId>truth</artifactId>
       <scope>test</scope>
     </dependency>
-    <dependency>
-      <groupId>io.opencensus</groupId>
-      <artifactId>opencensus-impl</artifactId>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>io.opencensus</groupId>
-      <artifactId>opencensus-testing</artifactId>
-      <scope>test</scope>
-    </dependency>
   </dependencies>
 </project>

google-http-client/src/main/java/com/google/api/client/http/HttpRequest.java

@@ -22,11 +22,6 @@
 import com.google.api.client.util.StreamingContent;
 import com.google.api.client.util.StringUtils;
 import com.google.common.util.concurrent.ThreadFactoryBuilder;
-import io.opencensus.common.Scope;
-import io.opencensus.contrib.http.util.HttpTraceAttributeConstants;
-import io.opencensus.trace.AttributeValue;
-import io.opencensus.trace.Span;
-import io.opencensus.trace.Tracer;
 import java.io.IOException;
 import java.io.InputStream;
 import java.util.Properties;
@@ -199,9 +194,6 @@ public final class HttpRequest {
   /** Sleeper. */
   private Sleeper sleeper = Sleeper.DEFAULT;
 
-  /** OpenCensus tracing component. */
-  private final Tracer tracer = OpenCensusUtils.getTracer();
-
   /**
    * Determines whether {@link HttpResponse#getContent()} of this request should return raw input
    * stream or not.
@@ -860,13 +852,7 @@ public HttpResponse execute() throws IOException {
     Preconditions.checkNotNull(requestMethod);
     Preconditions.checkNotNull(url);
 
-    Span span =
-        tracer
-            .spanBuilder(OpenCensusUtils.SPAN_NAME_HTTP_REQUEST_EXECUTE)
-            .setRecordEvents(OpenCensusUtils.isRecordEvent())
-            .startSpan();
     do {
-      span.addAnnotation("retry #" + (numRetries - retriesRemaining));
       // Cleanup any unneeded response from a previous iteration
       if (response != null) {
         response.ignore();
@@ -881,10 +867,6 @@ public HttpResponse execute() throws IOException {
       }
       // build low-level HTTP request
       String urlString = url.build();
-      addSpanAttribute(span, HttpTraceAttributeConstants.HTTP_METHOD, requestMethod);
-      addSpanAttribute(span, HttpTraceAttributeConstants.HTTP_HOST, url.getHost());
-      addSpanAttribute(span, HttpTraceAttributeConstants.HTTP_PATH, url.getRawPath());
-      addSpanAttribute(span, HttpTraceAttributeConstants.HTTP_URL, urlString);
 
       LowLevelHttpRequest lowLevelHttpRequest = transport.buildRequest(requestMethod, urlString);
       Logger logger = HttpTransport.LOGGER;
@@ -914,14 +896,11 @@ public HttpResponse execute() throws IOException {
       if (!suppressUserAgentSuffix) {
         if (originalUserAgent == null) {
           headers.setUserAgent(USER_AGENT_SUFFIX);
-          addSpanAttribute(span, HttpTraceAttributeConstants.HTTP_USER_AGENT, USER_AGENT_SUFFIX);
         } else {
           String newUserAgent = originalUserAgent + " " + USER_AGENT_SUFFIX;
           headers.setUserAgent(newUserAgent);
-          addSpanAttribute(span, HttpTraceAttributeConstants.HTTP_USER_AGENT, newUserAgent);
         }
       }
-      OpenCensusUtils.propagateTracingContext(span, headers);
 
       // headers
       HttpHeaders.serializeHeaders(headers, logbuf, curlbuf, logger, lowLevelHttpRequest);
@@ -1004,18 +983,8 @@ public HttpResponse execute() throws IOException {
       lowLevelHttpRequest.setTimeout(connectTimeout, readTimeout);
       lowLevelHttpRequest.setWriteTimeout(writeTimeout);
 
-      // switch tracing scope to current span
-      @SuppressWarnings("MustBeClosedChecker")
-      Scope ws = tracer.withSpan(span);
-      OpenCensusUtils.recordSentMessageEvent(span, lowLevelHttpRequest.getContentLength());
       try {
         LowLevelHttpResponse lowLevelHttpResponse = lowLevelHttpRequest.execute();
-        if (lowLevelHttpResponse != null) {
-          OpenCensusUtils.recordReceivedMessageEvent(span, lowLevelHttpResponse.getContentLength());
-          span.putAttribute(
-              HttpTraceAttributeConstants.HTTP_STATUS_CODE,
-              AttributeValue.longAttributeValue(lowLevelHttpResponse.getStatusCode()));
-        }
         // Flag used to indicate if an exception is thrown before the response is constructed.
         boolean responseConstructed = false;
         try {
@@ -1033,17 +1002,13 @@ public HttpResponse execute() throws IOException {
         if (!retryOnExecuteIOException
             && (ioExceptionHandler == null
                 || !ioExceptionHandler.handleIOException(this, retryRequest))) {
-          // static analysis shows response is always null here
-          span.end(OpenCensusUtils.getEndSpanOptions(null));
           throw e;
         }
         // Save the exception in case the retries do not work and we need to re-throw it later.
         executeException = e;
         if (loggable) {
           logger.log(Level.WARNING, "exception thrown while executing request", e);
         }
-      } finally {
-        ws.close();
       }
 
       // Flag used to indicate if an exception is thrown before the response has completed
@@ -1100,8 +1065,6 @@ public HttpResponse execute() throws IOException {
         }
       }
     } while (retryRequest);
-    span.end(OpenCensusUtils.getEndSpanOptions(response == null ? null : response.getStatusCode()));
-
     if (response == null) {
       // Retries did not help resolve the execute exception, re-throw it.
       throw executeException;
@@ -1218,12 +1181,6 @@ public HttpRequest setSleeper(Sleeper sleeper) {
     return this;
   }
 
-  private static void addSpanAttribute(Span span, String key, String value) {
-    if (value != null) {
-      span.putAttribute(key, AttributeValue.stringAttributeValue(value));
-    }
-  }
-
   private static String getVersion() {
     // attempt to read the library's version from a properties file generated during the build
     // this value should be read and cached for later use