core(network-request): exclude dns from rtt estimates#15110
core(network-request): exclude dns from rtt estimates#15110connorjclark wants to merge 1 commit intomainfrom
Conversation
|
I think this makes some sense. However our network throttling settings treat RTT as a property of the network as a whole, rather than a property of a single server. Even though we aren't connecting directly to the origin's server, that RTT time is still a necessary step for connecting to the origin. Perhaps the most accurate calculation would involve separating the DNS RTT from the actual server RTT but that probably isn't worth the effort. I can't say I'm swaying in one direction or the other at this point, but the difference seems to be minimal. |
(relevant code: The Simulator's I believe the Simulator's |
|
Updated branch, but let's wait for lantern trace database refresh. |
See a summary of how DNS works - at no point does it send a request to the actual server (that's the whole point of DNS - where is the server even?), so it doesn't make sense to include it in an estimate for server RTT.