Murtadha Hubail has submitted this change and it was merged. Change subject: [NO ISSUE][NET] Use Resolved Address When Parsing Net Address ......................................................................
[NO ISSUE][NET] Use Resolved Address When Parsing Net Address - user model changes: no - storage format changes: no - interface changes: no Details: - Use resolved addresses when parsing host/port to avoid unresolved address exception. Change-Id: I7b48647c9452ccfde44dcf0a06579502f59bc3d1 Reviewed-on: https://asterix-gerrit.ics.uci.edu/2808 Sonar-Qube: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Integration-Tests: Jenkins <[email protected]> Contrib: Jenkins <[email protected]> Reviewed-by: Murtadha Hubail <[email protected]> Reviewed-by: Michael Blow <[email protected]> Contrib: Michael Blow <[email protected]> --- M hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/NetworkUtil.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins: Verified; No violations found; ; Verified Michael Blow: Looks good to me, approved; No violations found Murtadha Hubail: Looks good to me, but someone else must approve diff --git a/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/NetworkUtil.java b/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/NetworkUtil.java index e6e21fa..c6b76fc 100644 --- a/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/NetworkUtil.java +++ b/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/NetworkUtil.java @@ -84,7 +84,7 @@ int lastColon = hostPortString.lastIndexOf(':'); String host = decodeIPv6LiteralHost(lastColon < 0 ? hostPortString : hostPortString.substring(0, lastColon)); int port = lastColon < 0 ? 0 : Integer.parseInt(hostPortString.substring(lastColon + 1)); - return InetSocketAddress.createUnresolved(host, port); + return new InetSocketAddress(host, port); } public static InetSocketAddress toInetSocketAddress(String maybeLiteralHost, int port) { -- To view, visit https://asterix-gerrit.ics.uci.edu/2808 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7b48647c9452ccfde44dcf0a06579502f59bc3d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]>
