[STORM-3550] Remove unused dependencies

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Won't Fix
• Component/s: storm-core
• Assignee: Unassigned
• Reporter: César Soto Valero
• Created: 03/Dec/19 09:32
• Updated: 23/Nov/23 11:57
• Resolved: 23/Nov/23 11:57

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)
. . .