[PR #615] [MERGED] add parallel query option #1553

New issue

Closed

opened 2026-03-16 02:12:43 +03:00 by kerem · 0 comments

kerem commented

2026-03-16 02:12:43 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hickory-dns/hickory-dns/pull/615
Author: @bluejekyll
Created: 11/14/2018
Status: ✅ Merged
Merged: 11/16/2018
Merged by: @bluejekyll

Base: master ← Head: par-requests

📝 Commits (3)

94d6c76 add parallel query option
f178b8b refactor NameServerPool to own ConnectionProvider
7895e9d add concurrency tests for ConnectionPool

📊 Changes

5 files changed (+416 additions, -62 deletions)

View changed files

📝 crates/resolver/CHANGELOG.md (+4 -0)
📝 crates/resolver/src/config.rs (+6 -0)
📝 crates/resolver/src/name_server_pool.rs (+106 -41)
📝 tests/integration-tests/src/mock_client.rs (+31 -6)
📝 tests/integration-tests/tests/name_server_pool_tests.rs (+269 -15)

📄 Description

This allow the NameserverPool to be configured to execute requests in parallel. By default parallel requests will be set to 2

@hawkw, I think this is something you might be interested in, as you may want to disable this? Though it could also help.

I'm currently working on a test to validate this code.

fixes #612

_{🔄 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/615 **Author:** [@bluejekyll](https://github.com/bluejekyll) **Created:** 11/14/2018 **Status:** ✅ Merged **Merged:** 11/16/2018 **Merged by:** [@bluejekyll](https://github.com/bluejekyll) **Base:** `master` ← **Head:** `par-requests` --- ### 📝 Commits (3) - [`94d6c76`](https://github.com/hickory-dns/hickory-dns/commit/94d6c7634f952324d38bdbb616c187da2c55b50d) add parallel query option - [`f178b8b`](https://github.com/hickory-dns/hickory-dns/commit/f178b8bb2c803295c1ef889b5fc13ea300668876) refactor NameServerPool to own ConnectionProvider - [`7895e9d`](https://github.com/hickory-dns/hickory-dns/commit/7895e9d6b63f130407affff9e8a919f5c8440914) add concurrency tests for ConnectionPool ### 📊 Changes **5 files changed** (+416 additions, -62 deletions) <details> <summary>View changed files</summary> 📝 `crates/resolver/CHANGELOG.md` (+4 -0) 📝 `crates/resolver/src/config.rs` (+6 -0) 📝 `crates/resolver/src/name_server_pool.rs` (+106 -41) 📝 `tests/integration-tests/src/mock_client.rs` (+31 -6) 📝 `tests/integration-tests/tests/name_server_pool_tests.rs` (+269 -15) </details> ### 📄 Description This allow the NameserverPool to be configured to execute requests in parallel. By default parallel requests will be set to 2 @hawkw, I think this is something you might be interested in, as you may want to disable this? Though it could also help. I'm currently working on a test to validate this code. fixes #612 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>