Upgrade to Netty 4 about nifty HOT 21 CLOSED

Is already being worked on

from nifty.

Sweet. Can't find that branch though.

from nifty.

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.

from nifty.

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.

from nifty.

Agreed.

from nifty.

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.

from nifty.

https://github.com/andrewcox/nifty/tree/netty4
https://github.com/andrewcox/swift/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.

from nifty.

notice pom netty dependency version is 4.0.0.CR1 ,have you test netty latest version,such as netty-4.0.12 final?

from nifty.

have Upgraded to Netty4 ?

from nifty.

https://github.com/andrewcox/nifty/tree/netty4
this branch is not using the stable version of netty 4 which is quite different and will lead the "java.lang.OutOfMemoryError: Java heap space" error.

from nifty.

@andrewcox are you still working on this ?
Seems there is no replacement solutions for using thrift on netty4 or netty5.

from nifty.

Still no plan for netty 4 yet?

from nifty.

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.

https://github.com/andrewcox/nifty/tree/netty4.0.24

from nifty.

And also needs to be rebased to current changes

from nifty.

Any updates on finalizing the Netty 4 branch soon?

from nifty.

Anyone working on 4.1 ?

from nifty.

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.

from nifty.

Any updates on this?

from nifty.

any updates?

from nifty.

any updates on this?

from nifty.

any updates on this?

from nifty.