davidB / metrics-influxdb

A reporter for metrics which announces measurements to an InfluxDB server.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Support for DNS name entry changes in UdpInlineSender

gmalouf opened this issue · comments

If the desire is for this client to be able to work with services that dynamically change their DNS name entries (many in AWS, GCP, etc), we need to update UdpInlineSender.

Inn UdpInlineSender, recreating the InetSocketAddress passed in to channel.send would allow for dynamic DNS resolution.

In the HTTP sender, I believe we are in good shape. The URL class's DNS lookup functionality (as strange as this is) is only used for equals and hashcode...hence opening each new connection as the code in this library does should result in fresh DNS lookups against JVM cache.