yaooqinn commented on a change in pull request #1320:
URL: https://github.com/apache/incubator-kyuubi/pull/1320#discussion_r740762891
##########
File path: docs/deployment/high_availability_guide.md
##########
@@ -29,9 +29,9 @@ Load balancing aims to optimize all Kyuubi service units
usage, maximize through
-With Hive JDBC Driver, a client can specify service discovery mode in JDBC
connection string, i.e. `serviceDiscoveryMode=zooKeeper;` and set
`zooKeeperNameSpace=kyuubiserver;`, then it can randomly pick one of the Kyuubi
service uris from the specified ZooKeeper address in the `/kyuubiserver` path.
+With Hive JDBC Driver, a client can specify service discovery mode in JDBC
connection string, i.e. `serviceDiscoveryMode=zooKeeper;` and set
`zooKeeperNamespace=kyuubi;`, then it can randomly pick one of the Kyuubi
service uris from the specified ZooKeeper address in the `/kyuubi` path.
-When we set `kyuubi.ha.enabled` to `true`, load balance mode is activated by
default. Please make sure that you specify the correct ZooKeeper address via
`kyuubi.ha.zookeeper.quorum` and `kyuubi.ha.zookeeper.client.port`.
+When we specify the ZooKeeper address via `kyuubi.ha.zookeeper.quorum`, load
balance mode is activated by default. Please make sure that you specify the
correct ZooKeeper address via `kyuubi.ha.zookeeper.quorum` and
`kyuubi.ha.zookeeper.client.port`.
Review comment:
Can we also update L38
--
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]
