anmolnar opened a new pull request, #5370: URL: https://github.com/apache/hbase/pull/5370
Looks like HBase has 2 different prefixes for ZooKeeper properties: `hbase.zookeeper.property.` `hbase.client.zookeeper.property.` and we also have these: `hbase.zookeeper.quorum` `hbase.client.zookeeper.quorum` Unfortunately I couldn't decrypt the meaning of each of every one of them, but we'd like to set system properties here which will be effective for all ZooKeeper clients in the same java process. Hence I only added the following properties which will be set in both client and server modes. ``` hbase.zookeeper.property.client.secure hbase.zookeeper.property.clientCnxnSocket hbase.zookeeper.property.ssl.keyStore.location hbase.zookeeper.property.ssl.keyStore.password hbase.zookeeper.property.ssl.keyStore.passwordPath hbase.zookeeper.property.ssl.trustStore.location hbase.zookeeper.property.ssl.trustStore.password hbase.zookeeper.property.ssl.trustStore.passwordPath ``` Please also note that *passwordPath properties are supported only from ZooKeeper 3.8 (stable version today), but HBase is still on 3.5 meaning these properties are currently ineffective with the default setup. Doc question: shall I add documentation in this patch or separately? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
