commit 17de353136950b43d253d24aae2b6ff947bdbe61
author Bipin Prasad <bipin_prasad@yahoo.com> Fri Mar 04 09:55:54 2022 -0800
committer Bipin Prasad <bprasad@yahooinc.com> Fri Mar 04 10:02:39 2022 -0800
tree aabd0584599db2703bd37eb271079d8290d72291
parent ea29fde13258517ff9a8229676802e8eaf19730d

[STORM-3829] Remove log4j 1.2.17 as a test dependency (#3450)

Co-authored-by: Bipin Prasad <bprasad@yahooinc.com>

storm-core/pom.xml

diff --git a/storm-core/pom.xml b/storm-core/pom.xml
index 162abc8..f54d4e0 100644
--- a/storm-core/pom.xml
+++ b/storm-core/pom.xml
@@ -58,17 +58,6 @@
             <scope>test</scope>
         </dependency>
 
-        <!--Hadoop Mini Cluster cannot use log4j2 bridge,
-        Surefire has a way to exclude the conflicting log4j API jar
-        from the classpath, classpathDependencyExcludes, but it didn't work in practice.
-        This is here as a work around to place it at the beginning of the classpath
-        even though maven does not officially support ordering of the classpath.-->
-        <dependency>
-            <groupId>log4j</groupId>
-            <artifactId>log4j</artifactId>
-            <version>1.2.17</version>
-            <scope>test</scope>
-        </dependency>
         <!--clojure-->
         <dependency>
             <groupId>org.clojure</groupId>