Uploaded image for project: 'Apache Storm'
  1. Apache Storm
  2. STORM-3829

Remove log4j version 1 (1.2.17) from storm-core

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 2.3.0
    • 2.4.0
    • storm-core
    • None

    Description

      storm-core/pom.xml has the following entry:

              <!--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>
      

      This dependency can be removed. "mvn test" goes to success. And "mvn dependency:tree | grep log4j" shows no transitive dependency on log4j-1.2.17.

      Attachments

        Issue Links

          Activity

            People

              bipinprasad Bipin Prasad
              bipinprasad Bipin Prasad
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 40m
                  40m