yaooqinn commented on code in PR #4649:
URL: https://github.com/apache/kyuubi/pull/4649#discussion_r1155425449


##########
docs/client/jdbc/kyuubi_jdbc.rst:
##########
@@ -127,34 +123,101 @@ accessing. The following is the format of the connection 
URL for the Kyuubi Hive
    - Properties are case-sensitive
    - Do not duplicate properties in the connection URL
 
-Connection URL over Http
+Connection URL over HTTP
 ************************
 
 .. versionadded:: 1.6.0
 
-.. code-block:: jdbc
+.. code-block::
 
-   
jdbc:subprotocol://host:port/schema;transportMode=http;httpPath=<http_endpoint>
+   jdbc:kyuubi://host:port/schema;transportMode=http;httpPath=<http_endpoint>
 
 - http_endpoint is the corresponding HTTP endpoint configured by 
`kyuubi.frontend.thrift.http.path` at the server side.
 
 Connection URL over Service Discovery
 *************************************
 
-.. code-block:: jdbc
+.. code-block::
+
+   jdbc:kyuubi://<zookeeper 
quorum>/;serviceDiscoveryMode=zooKeeper;zooKeeperNamespace=kyuubi
+
+- zookeeper quorum is the corresponding zookeeper cluster configured by 
`kyuubi.ha.addresses` at the server side.
+- zooKeeperNamespace is the corresponding namespace configured by 
`kyuubi.ha.namespace` at the server side.
+
+Kerberos Authentication
+-----------------------
+
+.. versionadded:: 1.6.0
+
+Kyuubi JDBC driver implements the Kerberos authentication based on JAAS 
framework instead of `Hadoop UserGroupInformation`_,
+which means it does not forcibly rely on Hadoop dependencies to connect a 
Kerberized Kyuubi Server.

Review Comment:
   Why capitalize an adjective?  - Kerberized



##########
docs/client/jdbc/kyuubi_jdbc.rst:
##########
@@ -127,34 +123,101 @@ accessing. The following is the format of the connection 
URL for the Kyuubi Hive
    - Properties are case-sensitive
    - Do not duplicate properties in the connection URL
 
-Connection URL over Http
+Connection URL over HTTP
 ************************
 
 .. versionadded:: 1.6.0
 
-.. code-block:: jdbc
+.. code-block::
 
-   
jdbc:subprotocol://host:port/schema;transportMode=http;httpPath=<http_endpoint>
+   jdbc:kyuubi://host:port/schema;transportMode=http;httpPath=<http_endpoint>
 
 - http_endpoint is the corresponding HTTP endpoint configured by 
`kyuubi.frontend.thrift.http.path` at the server side.
 
 Connection URL over Service Discovery
 *************************************
 
-.. code-block:: jdbc
+.. code-block::
+
+   jdbc:kyuubi://<zookeeper 
quorum>/;serviceDiscoveryMode=zooKeeper;zooKeeperNamespace=kyuubi
+
+- zookeeper quorum is the corresponding zookeeper cluster configured by 
`kyuubi.ha.addresses` at the server side.
+- zooKeeperNamespace is the corresponding namespace configured by 
`kyuubi.ha.namespace` at the server side.
+
+Kerberos Authentication
+-----------------------
+
+.. versionadded:: 1.6.0
+
+Kyuubi JDBC driver implements the Kerberos authentication based on JAAS 
framework instead of `Hadoop UserGroupInformation`_,
+which means it does not forcibly rely on Hadoop dependencies to connect a 
Kerberized Kyuubi Server.
+
+Kyuubi JDBC driver supports different approaches to connect a Kerberized 
Kyuubi Server. First of all, please follow

Review Comment:
   ditto



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to