From 87117a934c17a63ba61e761633196f926d57713a Mon Sep 17 00:00:00 2001
From: Arjan Bal <arjansbal@google.com>
Date: Sat, 2 Dec 2023 21:42:21 +0530
Subject: [PATCH] Remove opencensus-impl dependency from gcs connector

---
 pom.xml | 35 +++++++++++++++++++++++++++++++++++
 1 file changed, 35 insertions(+)

diff --git a/pom.xml b/pom.xml
index b264895d60..50e6d5b807 100644
--- a/pom.xml
+++ b/pom.xml
@@ -597,6 +597,10 @@
           <artifactId>grpc-api</artifactId>
           <groupId>io.grpc</groupId>
         </exclusion>
+        <exclusion>
+          <groupId>io.opencensus</groupId>
+          <artifactId>opencensus-impl</artifactId>
+        </exclusion>
         <exclusion>
           <artifactId>grpc-census</artifactId>
           <groupId>io.grpc</groupId>
@@ -962,6 +966,37 @@
           </execution>
         </executions>
       </plugin>
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-enforcer-plugin</artifactId>
+        <version>3.4.1</version>
+        <executions>
+          <execution>
+            <id>enforce-banned-dependencies</id>
+            <goals>
+              <goal>enforce</goal>
+            </goals>
+            <configuration>
+              <rules>
+                <bannedDependencies>
+                  <excludes>
+                    <!-- Exclude Open Census implementations to disable Open Census stats and tracing.
+                     They are not used by the plugins. For tracing, Open Census spawns a new thread
+                     and loads a single global instance of a TracingComponent. CDAP uses a separate
+                     class loader for each plugin. This leads to spawning a new thread for each plugin
+                     with the Open Census dependency. As this thread keeps a reference to the class
+                     loader, it prevents the class loader from being garbage collected leading
+                     to leaks. If no implementation of TracingComponent is found at runtime,
+                     Open Census uses a NoopTraceComponent -->
+                    <exclude>io.opencensus:opencensus-impl</exclude>
+                  </excludes>
+                </bannedDependencies>
+              </rules>
+              <fail>true</fail>
+            </configuration>
+          </execution>
+        </executions>
+      </plugin>
     </plugins>
   </build>