You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It looks delay is a rolling sum of all the server requests' round trip times. I may be mistaken, but it doesn't make sense for the new median calculation, delay should just be the delay for each server instead. It would make sense for the old mean calculation since this sum divided by the number of server requests would be the mean.
Also, I was wondering why delay is not a portion of the round trip time? In the function Do, the delay is computed as the total round trip time for the request. It seems like delay should only be the difference between the midpoint (reported by the server) and time of receipt by the client. Or this could be approximated by dividing the round trip time by two. It seems like the calculation at this line is expecting the delay between the server's response and client receipt.
It looks
delay
is a rolling sum of all the server requests' round trip times. I may be mistaken, but it doesn't make sense for the new median calculation, delay should just be the delay for each server instead. It would make sense for the old mean calculation since this sum divided by the number of server requests would be the mean.roughtime/client/client.go
Line 314 in 064fa60
Also, I was wondering why
delay
is not a portion of the round trip time? In the functionDo
, the delay is computed as the total round trip time for the request. It seems like delay should only be the difference between themidpoint
(reported by the server) and time of receipt by the client. Or this could be approximated by dividing the round trip time by two. It seems like the calculation at this line is expecting the delay between the server's response and client receipt.roughtime/client/client.go
Line 326 in 064fa60
The text was updated successfully, but these errors were encountered: