alibaba / jstorm

Enterprise Stream Process Engine
http://jstorm.io
Apache License 2.0
3.92k stars 1.8k forks source link

Remove unused dependencies #681

Open Celebrate-future opened 3 years ago

Celebrate-future commented 3 years ago

@Longda-Feng Hi, I am a user of project com.alibaba.jstorm:jstorm-yarn:1.0-SNAPSHOT. I found that its pom file introduced 79 dependencies. However, among them, 16 libraries (20%) have not been used by your project (the redundant dependencies are listed below). This PR helps com.alibaba.jstorm:jstorm-yarn:1.0-SNAPSHOT lose weight :) I have tested the revised configuration in my local environment. It is safe to remove the unused libraries.

Best regards

Redundant dependencies----


com.sun.jersey:jersey-json:jar:1.9:compile
com.sun.xml.bind:jaxb-impl:jar:2.2.3-1:compile
org.codehaus.jackson:jackson-jaxrs:jar:1.9.13:compile
com.sun.jersey:jersey-core:jar:1.9:compile
javax.servlet.jsp:jsp-api:jar:2.1:runtime
com.sun.jersey:jersey-server:jar:1.9:compile
com.sun.jersey.contribs:jersey-guice:jar:1.9:compile
com.sun.jersey:jersey-client:jar:1.9:compile
com.google.code.findbugs:jsr305:jar:1.3.9:compile
javax.xml.bind:jaxb-api:jar:2.2.2:compile
javax.activation:activation:jar:1.1:compile
asm:asm:jar:3.1:compile
javax.inject:javax.inject:jar:1:compile
org.codehaus.jettison:jettison:jar:1.1:compile
javax.xml.stream:stax-api:jar:1.0-2:compile
javax.servlet:servlet-api:jar:2.5:compile
CLAassistant commented 3 years ago

CLA assistant check
All committers have signed the CLA.