Skip to content
This repository has been archived by the owner on Jun 29, 2021. It is now read-only.

Remove redundant dependencies #249

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Celebrate-future
Copy link

@Celebrate-future Celebrate-future commented Mar 11, 2021

Hi, I am a user of project org.apache.metamodel:MetaModel-hadoop:5.3.3-SNAPSHOT. I found that its pom file introduced 93 dependencies. However, among them, 9 libraries (9%) have not been used by your project (the redundant dependencies are listed below). Reduce these useless dependencies can help prevent conflicts between library versions. MeanWhile, it can minimize the total added size to projects. Finally, it can help enable advanced scenarios for users of your package.
This PR helps org.apache.metamodel:MetaModel-hadoop:5.3.3-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----


javax.servlet.jsp:jsp-api:jar:2.1:runtime
com.sun.jersey:jersey-servlet:jar:1.19:compile
org.apache.kerby:kerby-xdr:jar:1.0.1:compile
javax.servlet:javax.servlet-api:jar:3.1.0:compile
com.sun.jersey:jersey-core:jar:1.19:compile
javax.ws.rs:jsr311-api:jar:1.1.1:compile
com.sun.jersey:jersey-client:jar:1.19:compile
com.fasterxml.jackson.jaxrs:jackson-jaxrs-json-provider:jar:2.7.8:compile
com.fasterxml.jackson.jaxrs:jackson-jaxrs-base:jar:2.7.8:compile

@kaspersorensen
Copy link
Contributor

LGTM. I restarted the failed Travis job.

PS: No reason to @mention any particular contributor like you did here. We want the community to respond, not any particular individual.

@Celebrate-future
Copy link
Author

Celebrate-future commented Mar 14, 2021

LGTM. I restarted the failed Travis job.

PS: No reason to @mention any particular contributor like you did here. We want the community to respond, not any particular individual.

Ok, I will delete the specific @.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants