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

[jdbc] Reduce bundle size from 16MB to 300kB #7770

Merged
merged 1 commit into from
May 24, 2020

Conversation

wborn
Copy link
Member

@wborn wborn commented May 24, 2020

This reduces the bundle size from 16MB to just 300kB by not embedding the feature bundle dependencies into the bundle itself.

Related to #5275

This reduces the bundle size from 16MB to just 300kB by not embedding the feature bundle dependencies into the bundle itself.

Signed-off-by: Wouter Born <github@maindrain.net>
@wborn wborn requested a review from a team as a code owner May 24, 2020 11:42
Copy link
Member

@kaikreuzer kaikreuzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds awesome, thanks :-)

@kaikreuzer
Copy link
Member

Just that the build does not compile anymore due to the missing dependencies...

@wborn
Copy link
Member Author

wborn commented May 24, 2020

According to the console output the build was a great success! 😄

@kaikreuzer
Copy link
Member

Yeah, despite the fact that it shows a red status and here also logged some errors: https://ci.openhab.org/job/PR-openHAB-Addons/145/org.openhab.addons.bundles$org.openhab.persistence.jdbc/console

@wborn wborn added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels May 24, 2020
@wborn wborn requested a review from kaikreuzer May 24, 2020 15:40
@kaikreuzer
Copy link
Member

Ok, you win ;-)

@kaikreuzer kaikreuzer merged commit ec0db5a into openhab:master May 24, 2020
@wborn wborn deleted the jdbc-reduce-bundle-size branch May 24, 2020 16:00
@cpmeister cpmeister added this to the 3.0 milestone May 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants