fix(gax): unconditionally wrap logging tracer

Author: westarle

gax-java/gax/src/main/java/com/google/api/gax/rpc/ClientContext.java

@@ -281,13 +281,11 @@ public static ClientContext create(StubSettings settings) throws IOException {
     if (apiTracerFactory instanceof SpanTracerFactory) {
       apiTracerFactory = apiTracerFactory.withContext(apiTracerContext);
     }
-    if (com.google.api.gax.logging.LoggingUtils.isLoggingEnabled()) {
-      com.google.api.gax.tracing.ApiTracerFactory loggingTracerFactory =
-          new com.google.api.gax.tracing.LoggingTracerFactory().withContext(apiTracerContext);
-      apiTracerFactory =
-          new com.google.api.gax.tracing.CompositeTracerFactory(
-              apiTracerFactory, loggingTracerFactory);
-    }
+    com.google.api.gax.tracing.ApiTracerFactory loggingTracerFactory =
+        new com.google.api.gax.tracing.LoggingTracerFactory().withContext(apiTracerContext);
+    apiTracerFactory =
+        new com.google.api.gax.tracing.CompositeTracerFactory(
+            apiTracerFactory, loggingTracerFactory);
 
     return newBuilder()
         .setBackgroundResources(backgroundResources.build())

gax-java/gax/src/main/java/com/google/api/gax/tracing/CompositeTracer.java

@@ -52,7 +52,8 @@ public Scope inScope() {
       scopes[i] = tracers.get(i).inScope();
     }
     return () -> {
-      for (Scope scope : scopes) {
+      for (int i = scopes.length - 1; i >= 0; i--) {
+        Scope scope = scopes[i];
         if (scope != null) {
           scope.close();
         }
@@ -62,22 +63,22 @@ public Scope inScope() {
 
   @Override
   public void operationSucceeded() {
-    for (ApiTracer tracer : tracers) {
-      tracer.operationSucceeded();
+    for (int i = tracers.size() - 1; i >= 0; i--) {
+      tracers.get(i).operationSucceeded();
     }
   }
 
   @Override
   public void operationCancelled() {
-    for (ApiTracer tracer : tracers) {
-      tracer.operationCancelled();
+    for (int i = tracers.size() - 1; i >= 0; i--) {
+      tracers.get(i).operationCancelled();
     }
   }
 
   @Override
   public void operationFailed(Throwable error) {
-    for (ApiTracer tracer : tracers) {
-      tracer.operationFailed(error);
+    for (int i = tracers.size() - 1; i >= 0; i--) {
+      tracers.get(i).operationFailed(error);
     }
   }