[ 
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)

Reply via email to