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

Remove unused dependencies

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: storm-core
    • Labels:
      None

      Description

      There are some dependencies declared in storm-core that are not used. These dependencies should be removed to make the core library slimmer and its dependency tree less complex, which helps to maintain the project.

      Unused dependencies with scope compile

      • joda-time:joda-time
      • org.eclipse.jetty:jetty-util
      • org.eclipse.jetty:jetty-servlet
      • org.eclipse.jetty:jetty-servlets
      • com.fasterxml.jackson.core:jackson-core
      • com.fasterxml.jackson.dataformat:jackson-dataformat-smile
      • commons-fileupload:commons-fileupload
      • commons-codec:commons-codec

      Unused dependencies with scope test

      • org.hamcrest:java-hamcrest

      PS: For some of these dependencies, such as jackson-core and commons:codec, I noticed that the same version is declared and included transitively in the dependency tree. This is redundant and should be avoided. The following is an excerpt of the Maven dependency tree that illustrates this situation.

      +- com.fasterxml.jackson.core:jackson-core:jar:2.9.8:compile 
      +- com.fasterxml.jackson.dataformat:jackson-dataformat-smile:jar:2.9.8:compile 
      |  \- (com.fasterxml.jackson.core:jackson-core:jar:2.9.8:compile - omitted for duplicate)
      
      +- commons-codec:commons-codec:jar:1.11:compile 
      +- org.apache.hadoop:hadoop-auth:jar:2.8.5:compile
      |  +- (commons-codec:commons-codec:jar:1.11:compile - omitted for duplicate)
      . . .
      

       

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              cesarsotovalero César Soto Valero
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: