[PR #1784] [MERGED] Order name servers by SRTT #2598

New issue

Closed

opened 2026-03-16 10:44:00 +03:00 by kerem · 0 comments

kerem commented

2026-03-16 10:44:00 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hickory-dns/hickory-dns/pull/1784
Author: @nhurley3
Created: 9/29/2022
Status: ✅ Merged
Merged: 10/10/2022
Merged by: @bluejekyll

Base: main ← Head: srtt-order

📝 Commits (7)

bf500b3 Order name servers by SRTT
0c0f72d Fix formatting issues
7bd6fa3 Provide an f64 total_cmp implementation
b421dc3 Add additional SRTT documentation
8a62ae1 Merge branch 'main' into srtt-order
44faeb3 Fix formatting
7bf15a6 Consistenly use Duration for NameServerStats

📊 Changes

4 files changed (+297 additions, -79 deletions)

View changed files

📝 Cargo.lock (+1 -0)
📝 crates/resolver/Cargo.toml (+2 -0)
📝 crates/resolver/src/name_server/name_server.rs (+6 -14)
📝 crates/resolver/src/name_server/name_server_stats.rs (+288 -65)

📄 Description

Calculates the smoothed round-trip time (SRTT) using an exponentially weighted moving average and orders the name servers by the value. The implemented algorithm is similar to what was discussed in #1702.

Fixes #1702 (originally filed by @peterthejohnston)

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/hickory-dns/hickory-dns/pull/1784 **Author:** [@nhurley3](https://github.com/nhurley3) **Created:** 9/29/2022 **Status:** ✅ Merged **Merged:** 10/10/2022 **Merged by:** [@bluejekyll](https://github.com/bluejekyll) **Base:** `main` ← **Head:** `srtt-order` --- ### 📝 Commits (7) - [`bf500b3`](https://github.com/hickory-dns/hickory-dns/commit/bf500b347ea6cdbb5782d790a945d6009aaab811) Order name servers by SRTT - [`0c0f72d`](https://github.com/hickory-dns/hickory-dns/commit/0c0f72d7a1b10784604aae688f1ac2dc37df6097) Fix formatting issues - [`7bd6fa3`](https://github.com/hickory-dns/hickory-dns/commit/7bd6fa386a344fc4c1695e85524bc17c2c453c27) Provide an f64 total_cmp implementation - [`b421dc3`](https://github.com/hickory-dns/hickory-dns/commit/b421dc3dc36eba7844de0f3b55dbdf2ad80e6bf3) Add additional SRTT documentation - [`8a62ae1`](https://github.com/hickory-dns/hickory-dns/commit/8a62ae1ca017daead47d08f4f894aa81ce7763ce) Merge branch 'main' into srtt-order - [`44faeb3`](https://github.com/hickory-dns/hickory-dns/commit/44faeb3c825d0072d87cd4ef08119de9ff7eb340) Fix formatting - [`7bf15a6`](https://github.com/hickory-dns/hickory-dns/commit/7bf15a663467ee22e5d9e2641ede2ca0d2527a3d) Consistenly use Duration for NameServerStats ### 📊 Changes **4 files changed** (+297 additions, -79 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+1 -0) 📝 `crates/resolver/Cargo.toml` (+2 -0) 📝 `crates/resolver/src/name_server/name_server.rs` (+6 -14) 📝 `crates/resolver/src/name_server/name_server_stats.rs` (+288 -65) </details> ### 📄 Description Calculates the smoothed round-trip time (SRTT) using an exponentially weighted moving average and orders the name servers by the value. The implemented algorithm is similar to what was discussed in #1702. Fixes #1702 (originally filed by @peterthejohnston) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>