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.

Sign up for GitHub

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

Jump to bottom

Epoll isn't used anymore #1301

Closed
davidlar opened this issue Jun 11, 2022 · 5 comments · Fixed by #1302
Closed

Epoll isn't used anymore #1301

davidlar opened this issue Jun 11, 2022 · 5 comments · Fixed by #1302
Labels
bug Something isn't working

Comments

@davidlar
Copy link
Contributor

After #1255 epoll (and probably kqueue) isn't used anymore, so nio will be used instead.
The problem is that the platform dependent jar files is not among the dependencies.

If I add "io.netty" % "netty-transport-native-epoll" % "4.1.77.Final" classifier "linux-x86_64" it works again for me.

Regards David
@davidlar davidlar added the bug Something isn't working label Jun 11, 2022
@tusharmath
Copy link
Collaborator

@davidlar Thanks for reporting this. This explains why there is a bit of a drop in performance in the latest version.

Would you be able to create a PR for this? Thanks once again!

@davidlar
Copy link
Contributor Author

Sure! I'll do that!

@davidlar
Copy link
Contributor Author

Do you want the PR in master and/or series/2 ?

@tusharmath
Copy link
Collaborator

Master

@davidlar davidlar mentioned this issue Jun 12, 2022
Merged
@davidlar
Copy link
Contributor Author

#1302 created

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix: Native EPoll and KQueue Dependencies davidlar/zio-http
2 participants
@tusharmath @davidlar