[PR #923] [MERGED] lock prevent the resolver to do concurrent requests #1795

Closed
opened 2026-03-16 02:25:44 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/hickory-dns/hickory-dns/pull/923
Author: @ntrippar
Created: 11/15/2019
Status: Merged
Merged: 11/16/2019
Merged by: @bluejekyll

Base: masterHead: concurrencylock


📝 Commits (2)

  • 431d0d4 lock prevent the nameserver to do concurrent request
  • 4eba9b2 Fix minor formatting error

📊 Changes

1 file changed (+15 additions, -12 deletions)

View changed files

📝 crates/resolver/src/name_server/name_server_pool.rs (+15 -12)

📄 Description

the lock on conns was preventing to make concurrent requests, in this fix i added the lock inside a scope so as soon the scope is done the conn will be dropped and the mutex will be unlocked.


🔄 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/923 **Author:** [@ntrippar](https://github.com/ntrippar) **Created:** 11/15/2019 **Status:** ✅ Merged **Merged:** 11/16/2019 **Merged by:** [@bluejekyll](https://github.com/bluejekyll) **Base:** `master` ← **Head:** `concurrencylock` --- ### 📝 Commits (2) - [`431d0d4`](https://github.com/hickory-dns/hickory-dns/commit/431d0d44a3247e53fbcf3be7df43d92ed8dfd3c6) lock prevent the nameserver to do concurrent request - [`4eba9b2`](https://github.com/hickory-dns/hickory-dns/commit/4eba9b2813c261b42fd9e0f177d357ebe4a6b027) Fix minor formatting error ### 📊 Changes **1 file changed** (+15 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `crates/resolver/src/name_server/name_server_pool.rs` (+15 -12) </details> ### 📄 Description the lock on conns was preventing to make concurrent requests, in this fix i added the lock inside a scope so as soon the scope is done the conn will be dropped and the mutex will be unlocked. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-16 02:25:44 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/hickory-dns#1795
No description provided.