Skip to content

[SCB-1763] upgrade netty and tcnative#1567

Merged
liubao68 merged 3 commits intoapache:masterfrom
chenxingwei-wayne:master
Feb 12, 2020
Merged

[SCB-1763] upgrade netty and tcnative#1567
liubao68 merged 3 commits intoapache:masterfrom
chenxingwei-wayne:master

Conversation

@chenxingwei-wayne
Copy link
Copy Markdown
Contributor

Follow this checklist to help us incorporate your contribution quickly and easily:

  • Make sure there is a JIRA issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a JIRA issue. Your pull request should address just this issue, without pulling in other changes.
  • Each commit in the pull request should have a meaningful subject line and body.
  • Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Run mvn clean install -Pit to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

@chenxingwei-wayne chenxingwei-wayne changed the title Fix netty issues by updating netty version to 4.1.45.Final and related tcnative version Netty shouldn't allow the invalid fold: https://github.com/netty/netty/issues/9866 Fix netty issues by updating netty version to 4.1.45.Final and related tcnative version Feb 11, 2020
Comment on lines 534 to 535
* Netty/Transport/Native/Epoll (io.netty:netty-transport-native-epoll:4.1.36.Final - http://netty.io/netty-transport-native-epoll/)
* Netty/Transport/Native/Unix/Common (io.netty:netty-transport-native-unix-common:4.1.36.Final - http://netty.io/netty-transport-native-unix-common/)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please check if these two dependencies is used. If not used, delete it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

netty-transport-native-epoll exists in our pom file, but it was excluded by exclusion tag, so we can say we don't use it.
netty-transport-native-unix-common we don't use it.

@liubao68 liubao68 changed the title Netty shouldn't allow the invalid fold: https://github.com/netty/netty/issues/9866 Fix netty issues by updating netty version to 4.1.45.Final and related tcnative version [SCB-1763] upgrade netty and tcnative Feb 11, 2020
@liubao68 liubao68 merged commit 5384666 into apache:master Feb 12, 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.

2 participants