Skip to content

Commit

Permalink
Restore UnknownHostException with required
Browse files Browse the repository at this point in the history
With the introduction of  "dfs.namenode.edits.qjournals.resolution-required" (defaults to true) we can restore the original check for a thrown exception.  Also update the hostname to avoid incidental local matches during lookup.
  • Loading branch information
Steve Vaughan Jr committed Feb 21, 2023
1 parent 9e0f026 commit 1a2b49c
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@
import java.net.InetSocketAddress;
import java.net.URI;
import java.net.URISyntaxException;
import java.net.UnknownHostException;
import java.util.List;
import java.util.Map;
import java.util.Random;
Expand Down Expand Up @@ -191,8 +192,7 @@ public void testRecoverAfterDoubleFailures() throws Exception {
}

/**
* Expect an unresolved address if a hostname can't be resolved, but allowing the QJM to be
* configured.
* Expect {@link UnknownHostException} if a hostname can't be resolved.
*/
@Test
public void testUnresolvableHostName() throws Exception {
Expand Down

0 comments on commit 1a2b49c

Please sign in to comment.