Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pull Dependencies only once #17321

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Pull Dependencies only once #17321

wants to merge 5 commits into from

Conversation

shigarg1
Copy link
Contributor

Fixes #17014

As part of this, there will still be some copies but it significantly reduces the overall size.

Currently there can be three copies

  1. in lib
  2. from dist profile
  3. from bundle-contrib-exts profile

Distribution size before fix - 911M
Distribution size after fix - 603M

@abhishekagarwal87
Copy link
Contributor

Extensions don't access jars that are in the other extension folders. Have you considered that?

@shigarg1
Copy link
Contributor Author

Extensions don't access jars that are in the other extension folders. Have you considered that?

No, I did not know about this. Will they have access to jars in lib folder?

@abhishekagarwal87
Copy link
Contributor

Extensions don't access jars that are in the other extension folders. Have you considered that?

No, I did not know about this. Will they have access to jars in lib folder?

Yes

@shigarg1
Copy link
Contributor Author

@cryptoe can you please help to review this.

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

Successfully merging this pull request may close these issues.

Reduce the size of the druid tar.
2 participants