Skip to content
This repository has been archived by the owner on Jan 12, 2022. It is now read-only.

[Update] Add dns cache in udp mode to ease the pressure of dns server. #53

Closed
wants to merge 2 commits into from

Conversation

fatelei
Copy link

@fatelei fatelei commented Dec 10, 2015

When using udp mode, the client will resolve ip address from dns server per request. It brings great pressure for my dns server. So I change the code, when using udp mode, resolve the ip address firstly.

@msiebuhr
Copy link
Owner

Another important detail here - this does completely sidestep DNS TTL, which may be quite bad to do.

@msiebuhr
Copy link
Owner

Also: If load on your DNS is a big thing, what about a local caching daemon? That way there's no need to maintain a caching DNS client in this codebase.

Thinking about all the things we then need to get right (we basically need to allow all DNS-relevant settings to be passed into statsd-client and get their implementation correct), having users do it outside the statsd-client might be a better solution.

@fatelei fatelei closed this Oct 27, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants