bob-barrett commented on a change in pull request #9902:
URL: https://github.com/apache/kafka/pull/9902#discussion_r570072638
##########
File path: clients/src/test/java/org/apache/kafka/clients/NetworkClientTest.java
##########
@@ -81,6 +85,19 @@
private final NetworkClient clientWithNoExponentialBackoff =
createNetworkClient(reconnectBackoffMsTest);
private final NetworkClient clientWithStaticNodes =
createNetworkClientWithStaticNodes();
private final NetworkClient clientWithNoVersionDiscovery =
createNetworkClientWithNoVersionDiscovery();
+ private ArrayList<InetAddress> initialAddresses = new
ArrayList<>(Arrays.asList(
Review comment:
Catching the `UnknownHostException` makes it a bit annoying, I threw it
in a static block to make the compiler happy. Let me know if you think there's
something cleaner.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
