zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nikhil Bhide (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ZOOKEEPER-2814) Ignore space after comma in connection string
Date Thu, 24 Aug 2017 11:33:00 GMT

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-2814?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16139917#comment-16139917

Nikhil Bhide commented on ZOOKEEPER-2814:

Hi Admin,
As per the guidelines, I forked out new branch, made the changes and opened a new pull request.
Build was passed successfully, but I can not check in the changes. Can you please review the
changes and give me write access so that I can commit the changes to Zookeeper- branch 3.4
and close this issue.

> Ignore space after comma in connection string
> ---------------------------------------------
>                 Key: ZOOKEEPER-2814
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2814
>             Project: ZooKeeper
>          Issue Type: Bug
>    Affects Versions: 3.5.3
>            Reporter: Viliam Durina
>            Assignee: Nikhil Ashok Bhide
>            Priority: Minor
> I'm using the following connection string:
> {{,<space>}}
> However, I get:
> {{java.net.UnknownHostException: Name or service not known
> at java.net.Inet6AddressImpl.lookupAllHostAddr(Native Method)
> at java.net.InetAddress$2.lookupAllHostAddr(InetAddress.java:928)
> at java.net.InetAddress.getAddressesFromNameService(InetAddress.java:1323)
> at java.net.InetAddress.getAllByName0(InetAddress.java:1276)
> at java.net.InetAddress.getAllByName(InetAddress.java:1192)
> at java.net.InetAddress.getAllByName(InetAddress.java:1126)
> at org.apache.zookeeper.client.StaticHostProvider.<init>(StaticHostProvider.java:61)
> at org.apache.zookeeper.ZooKeeper.<init>(ZooKeeper.java:445)
> at ...
> ...}}
> The problem was the space after the comma. I suggest to either make the space optional
or produce error on it, as this is a real pain to spot. Using the space also makes the connect
string more readable. Spaces are not allowed in domain names.

This message was sent by Atlassian JIRA

View raw message