Skip to content

Upgrade and shade netty #422

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

Merged
merged 1 commit into from
Nov 8, 2017
Merged

Upgrade and shade netty #422

merged 1 commit into from
Nov 8, 2017

Conversation

lutovich
Copy link
Contributor

Make driver depend on netty-handler instead of netty-all because it is much smaller and contains only features we use. Also netty version is upgraded from 4.1.15 to 4.1.16.

Introduced shade plugin to pack netty dependency inside driver jar. Shaded classes will be relocated to org.neo4j.driver.internal.shaded package. This means driver will not bring netty as it's transitive dependency and has it's private copy of netty classes removing the possibility of dependency conflicts.

@lutovich lutovich force-pushed the 1.5-shade-netty branch 2 times, most recently from cd9d9e9 to 92fe543 Compare October 19, 2017 22:18
Make driver depend on netty-handler instead of netty-all because it
is much smaller and contains only features we use. Also netty version
is upgraded from 4.1.15 to 4.1.16.

Introduced shade plugin to pack netty dependency inside driver jar.
Shaded classes will be relocated to `org.neo4j.driver.internal.shaded`
package. This means driver will not bring netty as it's transitive
dependency and has it's private copy of netty classes removing the
possibility of dependency conflicts.
@lutovich lutovich merged commit 8d180ce into neo4j:1.5 Nov 8, 2017
@lutovich lutovich deleted the 1.5-shade-netty branch November 8, 2017 11:46
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.

1 participant