[jira] [Commented] (ZOOKEEPER-2139) Zookeeper client configuration should not be java system property

["Gary Helmling (JIRA)" <ji...@apache.org>]

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

Gary Helmling commented on ZOOKEEPER-2139:
------------------------------------------

The reliance on global system properties for configuration can impact HBase replication setups when using ZooKeeper SASL auth, so I would love to see some resolution to this.  In this case, a single HBase cluster will need to connect with both a local ZK cluster and a remote ZK cluster for the replication peer.  If the two clusters require different SASL configurations, for example different values for zookeeper.sasl.client.username, then authentication with the peer cluster will fail.

Is the patch from June 30th the latest?  It seems to be missing the ClientConfiguration class.

> Zookeeper client configuration should not be java system property
> -----------------------------------------------------------------
>
>                 Key: ZOOKEEPER-2139
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2139
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: java client
>    Affects Versions: 3.5.0
>            Reporter: Surendra Singh Lilhore
>            Assignee: Surendra Singh Lilhore
>             Fix For: 3.5.2, 3.6.0
>
>         Attachments: ZOOKEEPER-2139.patch, ZOOKEEPER-2139.patch, ZOOKEEPER-2139_1.patch, ZOOKEEPER-2139_2.patch
>
>
> I have two ZK client in one JVM, one is secure client and second is normal client (For non secure cluster).
> "zookeeper.sasl.client" system property is "true" by default, because of this my second client connection is failing.
> We should pass all client configurations in client constructor like HDFS client.
> For example :
> {code}
> public ZooKeeper(String connectString, int sessionTimeout, Watcher watcher, Configuration conf) throws IOException
> 	{
> 		......
> 		......
> 	}
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)