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

winch: Cache kedge DNS resolution to optimize winch #62

Open
bwplotka opened this issue Sep 28, 2017 · 3 comments
Open

winch: Cache kedge DNS resolution to optimize winch #62

bwplotka opened this issue Sep 28, 2017 · 3 comments

Comments

@bwplotka
Copy link
Contributor

bwplotka commented Sep 28, 2017

We resolve all configured kEdges on demand. Even when it is used constantly, we perform DNS resolution every each time.

Let's optimize this. (DNS can be flaky)

@bwplotka bwplotka changed the title Cache kedge DNS resolution to optimize winch [Winch] Cache kedge DNS resolution to optimize winch Sep 28, 2017
@bwplotka bwplotka changed the title [Winch] Cache kedge DNS resolution to optimize winch winch: Cache kedge DNS resolution to optimize winch Jan 8, 2018
@ghost
Copy link

ghost commented Apr 18, 2018

Found the same thing with DNS. Especially on mobiles.
I was thinking of doing the DNS resolution in the golang code itself might help because you can then cache it.

@bwplotka
Copy link
Contributor Author

@gedw99 one thing is to perform DNS resolution less often (cache it), second is making it independently to the request.

Also, I think it is all in Golang. If you dial to some DNS name, golang client will use net.LookupXXX to find IP. However we can write something before client to poll DNS every X seconds and change current IP that will be used directly in dial to avoid internal Go client lookup on every request.

@ghost
Copy link

ghost commented Apr 18, 2018 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant