-
Notifications
You must be signed in to change notification settings - Fork 313
Upgrade to Netty 4 #50
Comments
Is already being worked on |
Sweet. Can't find that branch though. |
Sorry I should have been more specific. There is no branch for it yet, but it's almost ready to go up. Just working out a few performance problems. |
Most users of nifty / swift are still on netty 3.x. Unless we can ensure that netty 3 and netty 4 can co-exist, we need to publish two separate artifacts. I can a factoring out of a reasonable nifty API and then offer a n3 and n4 implementation but an unconditional switch would probably require shading in the netty dependency. |
Agreed. |
Netty 3 and 4 are in different packages (io.netty and org.jboss.netty) , so there should be no problem running them in the same VM. |
https://github.com/andrewcox/nifty/tree/netty4 These branches build and pass all tests running on netty 4, will be cleaning up and make pull requests soon after netty 4 final is released. |
notice pom netty dependency version is 4.0.0.CR1 ,have you test netty latest version,such as netty-4.0.12 final? |
have Upgraded to Netty4 ? |
https://github.com/andrewcox/nifty/tree/netty4 |
@andrewcox are you still working on this ? |
Still no plan for netty 4 yet? |
I am working on this again. Here is my working branch that builds & verifies but needs lots of addressing todo items and some code cleanup. |
And also needs to be rebased to current changes |
Any updates on finalizing the Netty 4 branch soon? |
Anyone working on 4.1 ? |
Thank you for reporting this issue and appreciate your patience. We've notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed. |
Any updates on this? |
any updates? |
any updates on this? |
1 similar comment
any updates on this? |
Netty 4 is nearing completion so, it would be lovely to see that up for use.
The text was updated successfully, but these errors were encountered: