-
Notifications
You must be signed in to change notification settings - Fork 471
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
[Performance] HickoryDNS loses nearly 50% of queries #2613
Comments
Hi @mokeyish, Running the same git commit as you, this is what I see against 1.1.1.1:
and 8.8.8.8:
Most of the few query failures I did see went away with a longer timeout (I was getting late reply messages in a few cases.) So, with DNS servers that are reasonably close to me, I can't duplicate the behavior you are seeing. If you can reproduce this readily, trace logs from hickory would be helpful. |
@marcus0x62 Are your two directly testing 1.1.1.1 and 8.8.8.8? One of my tests is to test 119.29.29.29 directly, and the other is to start HickoryDNS and set the upstream to 119.29.29.29. |
No, the output of dnsperf in my previous message shows requests to Hickory, configured to forward to 1.1.1.1 and 8.8.8.8. I used the same forwarding configuration as in your issue report. |
Your data is so close, is it because my computer performance is relatively low? I am running it in WSL. |
@marcus0x62 I ran it on cloudcone and got exactly the same result as yours. |
I think I'd need to see the full log output to venture any sort of guess as to what's going on. Trying running your server with:
and attach the full log file. |
Describe the bug
I made a simple DNS performance test repository and obtained the most popular 1 million domain names as a test set.
Then it was discovered that HickoryDNS actually lost nearly 50% of the queries.
To Reproduce
https://github.com/mokeyish/dnsperf-testing
Steps to reproduce the behavior:
Case 1: Use nameserver 119.29.29.29 directly
just test
Case 2: use HickoryDNS
.env
just test
Expected behavior
Fix this performance issue.
System:
Version:
Crate: server
Version: 001ced8
The text was updated successfully, but these errors were encountered: