-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Add netty 4 transport module #16
Comments
一些其它的三方库也可以考虑升级到最新,比如log4j2(还能配合disruptor)代替log4j,zkclient,Commons Pool2等等。 |
以后起码要三方模块要考虑升级下,有些包太老了 |
Netty4支持大概会什么时候放出来? |
@simpleSOA 目前我们没有遇到依赖版本太老导致的问题,如果觉得哪个依赖急需升级可以提issue或pr。 @sumory netty4的支持还在规划中,可能要过上一段时间。 |
log4j2自己可以添加支持,只需要在maven中把自带的log相关的jar排除,并增加log4j2相关的实现即可 |
“log4j的依赖我们未来考虑尽量使用slf4j管理而不关心具体实现” 这个设计思路是对的。 |
加入netty4的支持,有没有有这个想法的?现在在实施吗? |
Netty 3 is TOO old, we should add a module for netty 4.
The text was updated successfully, but these errors were encountered: