Add support for r2 Netty client DNS metrics, and clarify timeout errors for DNS timeouts.

linkedin / rest.li

Rest.li is a REST+JSON framework for building robust, scalable service architectures using dynamic discovery and simple asynchronous APIs.

rest.li

Other

2.51k stars 546 forks source link

Add support for r2 Netty client DNS metrics, and clarify timeout errors for DNS timeouts. #809

Closed jpstewart closed 2 years ago

jpstewart commented 2 years ago

There can be a misleading situation where DNS resolution is timing out due to host system DNS issues, which present as request timeouts in the error messages. In these situations the request is actually never made, since the DNS resolution is done within the same timeout boundaries as the request. This will clarify the error messages and also makes it possible to use JMX metrics to track DNS latency.