[jira] [Commented] (TINKERPOP-2289) Use address instead of hostname for connection

["Divij Vaidya (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TINKERPOP-2289?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16943117#comment-16943117 ] 

Divij Vaidya commented on TINKERPOP-2289:
-----------------------------------------

After this change, on my Mac OSX 10.14,  `InetAddress.getAllByName("localhost")` adds two IP addresses as two separate hosts. The IP addresses are IPv4 loopback 127.0.0.1 and IPv6 loopback :::1. This leads the tests to think that there are two servers available for the connections whereas in reality it is the same server.


This seems like an unintended side effect of this change. It might lead to dev env breaking for folks who have a computer where local name service resolves to both IPv6 and IPv4.

 

We should address this impact before shipping this PR.

> Use address instead of hostname for connection
> ----------------------------------------------
>
>                 Key: TINKERPOP-2289
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2289
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: driver
>    Affects Versions: 3.4.3
>            Reporter: Hua Jiang
>            Assignee: Stephen Mallette
>            Priority: Minor
>             Fix For: 3.5.0, 3.3.9, 3.4.4
>
>
> The current implementation uses hostname to create connections. It causes two problems in a scenario where multiple addresses are bound to the same domain name.
>  # Load balancing is broken because multiple `Host` may connect to the same address, and some addresses may have no connection at all.
>  # It produces misleading log messages because the address in the host label may not be the one the host is actually connected to.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)