Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support DNS cache and/or asynchronous resolution for master addresses #1000

Closed
ajcaldera1 opened this issue Mar 14, 2019 · 0 comments
Closed

Support DNS cache and/or asynchronous resolution for master addresses #1000

ajcaldera1 opened this issue Mar 14, 2019 · 0 comments
Assignees
@spolitov
Labels
kind/enhancement This is an enhancement of an existing feature

Comments

@ajcaldera1
Copy link
Contributor

In the event of short TTL for DNS names, we should either cache or asynchronously update DNS entries for masters to avoid excessive refetch or possible delays in resolution.

Example:


W0314 17:07:21.496300 26257 yb_rpc.cc:274] Call yb.tserver.TabletServerService.Read 35.241.224.95:49083 => 172.24.16.4:7100 (request call id 1508417) took 4176ms (client timeout 2499ms).
W0314 17:07:22.156210 26356 net_util.cc:192] Time spent resolving address for xxx-yug-202.xxxxxx.xx: real 0.253s    user 0.000s    sys 0.000s
W0314 17:07:22.425374 23827 net_util.cc:192] Time spent resolving address for xxx-yug-241.xxxxxx.xx: real 0.266s    user 0.000s    sys 0.000s
W0314 17:07:22.827749 23826 net_util.cc:192] Time spent resolving address for xxx-yug-221.xxxxxx.xx: real 0.275s    user 0.000s    sys 0.000s
@ajcaldera1 ajcaldera1 added the kind/enhancement This is an enhancement of an existing feature label Mar 14, 2019
yugabyte-ci pushed a commit that referenced this issue Mar 30, 2019
@spolitov
ENG-4872: #1000: Parallel DNS resolution while filling system.peers a…
4b48e25 
…nd system.local tables

Summary:
While serving request to system.peers or system.local tables we are doing DNS resolution,
to convert hostname to IP address.
This diff adds parallel DNS resolution in this scenario.
So when DNS resolution takes significant time, those times would not add.

Test Plan: Jenkins

Reviewers: mihnea, robert, mikhail

Reviewed By: mikhail

Subscribers: bogdan, ybase, bharat

Differential Revision: https://phabricator.dev.yugabyte.com/D6409
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@spolitov spolitov

Labels
kind/enhancement This is an enhancement of an existing feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@spolitov @ajcaldera1