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

Use Epoll datagram channel for DNS resolver group when running on Epoll #477

Merged
merged 3 commits into from Mar 25, 2019

Conversation

@kachayev
Copy link
Collaborator

kachayev commented Feb 4, 2019

Fixes #476.

It also extends DNS options with new :epoll? setting.

@alexander-yakushev

This comment has been minimized.

Copy link
Contributor

alexander-yakushev commented Feb 4, 2019

Verified that the patch works, thanks!

@kachayev

This comment has been minimized.

Copy link
Collaborator Author

kachayev commented Feb 17, 2019

@ztellman Quick update for the PR: getting rid of unnecessary event loop specification for the DNS resolver builder (instance of the appropriate event loop will be setup by the group internally).

@ztellman ztellman merged commit bedc468 into ztellman:master Mar 25, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.