[PR #2557] [MERGED] feat: Implement Round Robin server selection for DNS lookups #3136

New issue

Closed

opened 2026-03-16 11:27:22 +03:00 by kerem · 0 comments

kerem commented

2026-03-16 11:27:22 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hickory-dns/hickory-dns/pull/2557
Author: @hingbong
Created: 11/6/2024
Status: ✅ Merged
Merged: 11/11/2024
Merged by: @djc

Base: main ← Head: rr

📝 Commits (1)

0eeeaa3 feat: Implement Round Robin server selection for DNS lookups

📊 Changes

2 files changed (+52 additions, -14 deletions)

View changed files

📝 crates/resolver/src/config.rs (+3 -0)
📝 crates/resolver/src/name_server/name_server_pool.rs (+49 -14)

📄 Description

Adds the RoundRobin variant to the ServerOrderingStrategy enum.
Modifies send function to use AtomicUsize and perform Round Robin selection based on the number of concurrent requests.
Ensures at least one server is used even for num_concurrent_reqs = 1.

_{🔄 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/2557 **Author:** [@hingbong](https://github.com/hingbong) **Created:** 11/6/2024 **Status:** ✅ Merged **Merged:** 11/11/2024 **Merged by:** [@djc](https://github.com/djc) **Base:** `main` ← **Head:** `rr` --- ### 📝 Commits (1) - [`0eeeaa3`](https://github.com/hickory-dns/hickory-dns/commit/0eeeaa36063c78e35f72c052ec7762b1c9a4c3aa) feat: Implement Round Robin server selection for DNS lookups ### 📊 Changes **2 files changed** (+52 additions, -14 deletions) <details> <summary>View changed files</summary> 📝 `crates/resolver/src/config.rs` (+3 -0) 📝 `crates/resolver/src/name_server/name_server_pool.rs` (+49 -14) </details> ### 📄 Description * Adds the `RoundRobin` variant to the `ServerOrderingStrategy` enum. * Modifies `send` function to use AtomicUsize and perform Round Robin selection based on the number of concurrent requests. * Ensures at least one server is used even for `num_concurrent_reqs = 1`. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>