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

Remove log4j version 1 (1.2.17) from storm-core

Log workAgile BoardRank to TopRank to BottomAttach filesAttach ScreenshotBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    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

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            bipinprasad Bipin Prasad Assign to me
            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

              Slack

                Issue deployment