Skip to content
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

Why not netty for networking? #373

Closed
yrsh opened this issue Dec 10, 2020 · 5 comments
Closed

Why not netty for networking? #373

yrsh opened this issue Dec 10, 2020 · 5 comments
Assignees

Comments

@yrsh
Copy link

yrsh commented Dec 10, 2020

Hi!
Why the current implementation doesn't use netty?

@ColinSullivan1
Copy link
Member

@sasbury would have insight here..

@sasbury
Copy link
Contributor

sasbury commented Dec 18, 2020

Someone just ported to NIO, i am not sure that there is any performance benefit. As far as Netty goes I would say it is heavy dependency to add, but someone could build a version that uses that.

@gytis-ivaskevicius
Copy link

gytis-ivaskevicius commented Jan 15, 2021

Netty is not only reactive but also has "native transport" implementation, I'm no expert but if I recall correctly it instead of using NIO has some JNI magic that handles networking more directly which results in lower latency. TL;DR: I'd expect marginally but noticeably better performance.

I'd love to see it implemented as part of this project. Java world is slowly turning reactive and sooner or later it will need to be implemented and I'm quite sure of it

@sasbury
Copy link
Contributor

sasbury commented Jan 15, 2021

My concern is three fold. First, JNI brings new deployment issues to anyone using the library. Second, since this is a client library and not a server library many of the NIO type optimizations don't actually add much value. The library doesn't open lots of sockets, just the one to the server. Someone just did an NIO re-write, I am curious how it performs in comparison to old socket code. Netty itself brings a big dependency (with possible down stream dependencies).

@scottf
Copy link
Contributor

scottf commented Apr 30, 2021

Closing, added won't fix label. sasbury's comments pretty well sums it up, so no need to elaborate.

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

No branches or pull requests

6 participants