This is an automated email from the ASF dual-hosted git repository.

jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 3697552290 Remove commons-logging exclusion (#12730)
3697552290 is described below

commit 3697552290f64b62e938152b891764adfb0ee031
Author: Xiang Fu <xiangfu.1...@gmail.com>
AuthorDate: Tue Mar 26 13:54:14 2024 -0700

    Remove commons-logging exclusion (#12730)
---
 pinot-common/pom.xml                                     |  6 ------
 pinot-distribution/pom.xml                               |  4 ----
 pinot-integration-test-base/pom.xml                      |  4 ----
 pinot-integration-tests/pom.xml                          |  4 ----
 .../pinot-stream-ingestion/pinot-pulsar/pom.xml          |  6 ------
 pom.xml                                                  | 16 ++++++++++++++++
 6 files changed, 16 insertions(+), 24 deletions(-)

diff --git a/pinot-common/pom.xml b/pinot-common/pom.xml
index 65f9778177..b6e47114e0 100644
--- a/pinot-common/pom.xml
+++ b/pinot-common/pom.xml
@@ -193,12 +193,6 @@
     <dependency>
       <groupId>org.apache.httpcomponents</groupId>
       <artifactId>httpclient</artifactId>
-      <exclusions>
-        <exclusion>
-          <groupId>commons-logging</groupId>
-          <artifactId>commons-logging</artifactId>
-        </exclusion>
-      </exclusions>
     </dependency>
     <dependency>
       <groupId>org.apache.httpcomponents</groupId>
diff --git a/pinot-distribution/pom.xml b/pinot-distribution/pom.xml
index af5be1d237..a9bb9f5ecd 100644
--- a/pinot-distribution/pom.xml
+++ b/pinot-distribution/pom.xml
@@ -112,10 +112,6 @@
           <groupId>org.apache.hadoop</groupId>
           <artifactId>hadoop-mapreduce-client-core</artifactId>
         </exclusion>
-        <exclusion>
-          <groupId>commons-logging</groupId>
-          <artifactId>commons-logging</artifactId>
-        </exclusion>
       </exclusions>
     </dependency>
     <dependency>
diff --git a/pinot-integration-test-base/pom.xml 
b/pinot-integration-test-base/pom.xml
index 2c6eb3dd31..6f07a8f5d4 100644
--- a/pinot-integration-test-base/pom.xml
+++ b/pinot-integration-test-base/pom.xml
@@ -60,10 +60,6 @@
           <groupId>org.slf4j</groupId>
           <artifactId>slf4j-log4j12</artifactId>
         </exclusion>
-        <exclusion>
-          <groupId>commons-logging</groupId>
-          <artifactId>commons-logging</artifactId>
-        </exclusion>
       </exclusions>
     </dependency>
     <dependency>
diff --git a/pinot-integration-tests/pom.xml b/pinot-integration-tests/pom.xml
index 1db7433c3f..3280f652ca 100644
--- a/pinot-integration-tests/pom.xml
+++ b/pinot-integration-tests/pom.xml
@@ -196,10 +196,6 @@
           <groupId>org.slf4j</groupId>
           <artifactId>slf4j-log4j12</artifactId>
         </exclusion>
-        <exclusion>
-          <groupId>commons-logging</groupId>
-          <artifactId>commons-logging</artifactId>
-        </exclusion>
       </exclusions>
     </dependency>
     <dependency>
diff --git a/pinot-plugins/pinot-stream-ingestion/pinot-pulsar/pom.xml 
b/pinot-plugins/pinot-stream-ingestion/pinot-pulsar/pom.xml
index eaaf786729..8ef56f74ff 100644
--- a/pinot-plugins/pinot-stream-ingestion/pinot-pulsar/pom.xml
+++ b/pinot-plugins/pinot-stream-ingestion/pinot-pulsar/pom.xml
@@ -196,12 +196,6 @@
     <dependency>
       <groupId>org.apache.pinot</groupId>
       <artifactId>pinot-spi</artifactId>
-      <exclusions>
-        <exclusion>
-          <groupId>commons-logging</groupId>
-          <artifactId>commons-logging</artifactId>
-        </exclusion>
-      </exclusions>
     </dependency>
   </dependencies>
 </project>
diff --git a/pom.xml b/pom.xml
index 918ffb6d83..0d623d3d57 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1701,6 +1701,22 @@
               </goals>
               <phase>validate</phase>
             </execution>
+            <execution>
+              <id>enforce-banned-dependencies</id>
+              <goals>
+                <goal>enforce</goal>
+              </goals>
+              <configuration>
+                <rules>
+                  <bannedDependencies>
+                    <excludes>
+                      <exclude>commons-logging:commons-logging</exclude>
+                    </excludes>
+                  </bannedDependencies>
+                </rules>
+                <fail>true</fail> <!-- Fail the build if banned dependencies 
are found -->
+              </configuration>
+            </execution>
           </executions>
           <configuration>
             <rules>


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to