You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by ch...@apache.org on 2024/03/04 05:39:00 UTC

(kyuubi) branch master updated: [KYUUBI #6119] [DOC] Update doc for HA/Zookeeper Configuration

This is an automated email from the ASF dual-hosted git repository.

chengpan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kyuubi.git


The following commit(s) were added to refs/heads/master by this push:
     new 97d02a492 [KYUUBI #6119] [DOC] Update doc for HA/Zookeeper Configuration
97d02a492 is described below

commit 97d02a492d95991656037cbd00247dec9ad69cd4
Author: Manhua <ma...@apache.org>
AuthorDate: Mon Mar 4 05:38:52 2024 +0000

    [KYUUBI #6119] [DOC] Update doc for HA/Zookeeper Configuration
    
    # :mag: Description
    ## Issue References ๐Ÿ”—
    
    This pull request update doc for HA/Zookeeper Configuration
    
    ## Describe Your Solution ๐Ÿ”ง
    
    Replace conf name with the new ones and mark olds as deprecated
    
    ## Types of changes :bookmark:
    
    - [x] Bugfix (non-breaking change which fixes an issue)
    - [ ] New feature (non-breaking change which adds functionality)
    - [ ] Breaking change (fix or feature that would cause existing functionality to change)
    
    ## Test Plan ๐Ÿงช
    
    #### Behavior Without This Pull Request :coffin:
    
    #### Behavior With This Pull Request :tada:
    
    #### Related Unit Tests
    
    ---
    
    # Checklist ๐Ÿ“
    
    - [ ] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)
    
    **Be nice. Be informative.**
    
    Closes #6119 from kevinjmh/doc-update.
    
    Closes #6119
    
    49d0fffa0 [Manhua] [DOC] Update doc for HA/Zookeeper Configuration
    
    Authored-by: Manhua <ma...@apache.org>
    Signed-off-by: Cheng Pan <ch...@apache.org>
---
 docs/configuration/settings.md                                 |  6 +++---
 docs/connector/spark/delta_lake_with_azure_blob.rst            |  2 +-
 docs/deployment/high_availability_guide.md                     | 10 +++++-----
 docs/tools/kyuubi-ctl.rst                                      |  2 +-
 .../main/scala/org/apache/kyuubi/ha/HighAvailabilityConf.scala |  2 +-
 .../main/scala/org/apache/kyuubi/zookeeper/ZookeeperConf.scala |  4 ++--
 6 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/docs/configuration/settings.md b/docs/configuration/settings.md
index 37b79d6c3..a5af4e180 100644
--- a/docs/configuration/settings.md
+++ b/docs/configuration/settings.md
@@ -301,7 +301,7 @@ You can configure the Kyuubi properties in `$KYUUBI_HOME/conf/kyuubi-defaults.co
 | kyuubi.ha.etcd.ssl.client.key.path             | &lt;undefined&gt;                                              | Where the etcd SSL key file is stored.                                                                                                                                                                                   | string   | 1.6.0 |
 | kyuubi.ha.etcd.ssl.enabled                     | false                                                          | When set to true, will build an SSL secured etcd client.                                                                                                                                                                 | boolean  | 1.6.0 |
 | kyuubi.ha.namespace                            | kyuubi                                                         | The root directory for the service to deploy its instance uri                                                                                                                                                            | string   | 1.6.0 |
-| kyuubi.ha.zookeeper.acl.enabled                | false                                                          | Set to true if the ZooKeeper ensemble is kerberized                                                                                                                                                                      | boolean  | 1.0.0 |
+| kyuubi.ha.zookeeper.acl.enabled                | false                                                          | (deprecated) Set to true if the ZooKeeper ensemble is kerberized                                                                                                                                                         | boolean  | 1.0.0 |
 | kyuubi.ha.zookeeper.auth.digest                | &lt;undefined&gt;                                              | The digest auth string is used for ZooKeeper authentication, like: username:password.                                                                                                                                    | string   | 1.3.2 |
 | kyuubi.ha.zookeeper.auth.keytab                | &lt;undefined&gt;                                              | Location of the Kyuubi server's keytab that is used for ZooKeeper authentication.                                                                                                                                        | string   | 1.3.2 |
 | kyuubi.ha.zookeeper.auth.principal             | &lt;undefined&gt;                                              | Kerberos principal name that is used for ZooKeeper authentication.                                                                                                                                                       | string   | 1.3.2 |
@@ -512,11 +512,11 @@ You can configure the Kyuubi properties in `$KYUUBI_HOME/conf/kyuubi-defaults.co
 | kyuubi.zookeeper.embedded.client.use.hostname    | false              | When true, embedded Zookeeper prefer to bind hostname, otherwise, ip address.                                                                                                                                                              | boolean | 1.7.2 |
 | kyuubi.zookeeper.embedded.data.dir               | embedded_zookeeper | dataDir for the embedded zookeeper server where stores the in-memory database snapshots and, unless specified otherwise, the transaction log of updates to the database. If it is a relative path, it is resolved relative to KYUUBI_HOME. | string  | 1.2.0 |
 | kyuubi.zookeeper.embedded.data.log.dir           | embedded_zookeeper | dataLogDir for the embedded ZooKeeper server where writes the transaction log. If it is a relative path, it is resolved relative to KYUUBI_HOME.                                                                                           | string  | 1.2.0 |
-| kyuubi.zookeeper.embedded.directory              | embedded_zookeeper | The temporary directory for the embedded ZooKeeper server. If it is a relative path, it is resolved relative to KYUUBI_HOME.                                                                                                               | string  | 1.0.0 |
+| kyuubi.zookeeper.embedded.directory              | embedded_zookeeper | (deprecated) The temporary directory for the embedded ZooKeeper server. If it is a relative path, it is resolved relative to KYUUBI_HOME.                                                                                                  | string  | 1.0.0 |
 | kyuubi.zookeeper.embedded.max.client.connections | 120                | maxClientCnxns for the embedded ZooKeeper server to limit the number of concurrent connections of a single client identified by IP address                                                                                                 | int     | 1.2.0 |
 | kyuubi.zookeeper.embedded.max.session.timeout    | 60000              | maxSessionTimeout in milliseconds for the embedded ZooKeeper server will allow the client to negotiate. Defaults to 20 times the tickTime                                                                                                  | int     | 1.2.0 |
 | kyuubi.zookeeper.embedded.min.session.timeout    | 6000               | minSessionTimeout in milliseconds for the embedded ZooKeeper server will allow the client to negotiate. Defaults to 2 times the tickTime                                                                                                   | int     | 1.2.0 |
-| kyuubi.zookeeper.embedded.port                   | 2181               | The port of the embedded ZooKeeper server                                                                                                                                                                                                  | int     | 1.0.0 |
+| kyuubi.zookeeper.embedded.port                   | 2181               | (deprecated) The port of the embedded ZooKeeper server                                                                                                                                                                                     | int     | 1.0.0 |
 | kyuubi.zookeeper.embedded.tick.time              | 3000               | tickTime in milliseconds for the embedded ZooKeeper server                                                                                                                                                                                 | int     | 1.2.0 |
 
 ## Spark Configurations
diff --git a/docs/connector/spark/delta_lake_with_azure_blob.rst b/docs/connector/spark/delta_lake_with_azure_blob.rst
index bdca3f5df..67065641d 100644
--- a/docs/connector/spark/delta_lake_with_azure_blob.rst
+++ b/docs/connector/spark/delta_lake_with_azure_blob.rst
@@ -177,7 +177,7 @@ Add the following content:
    kyuubi.frontend.bind.host       <YOUR_HOST>
    kyuubi.frontend.bind.port       10009
    # If you use your own zk cluster, you need to configure your zk host port.
-   # kyuubi.ha.zookeeper.quorum    <YOUR_HOST>:2181
+   # kyuubi.ha.addresses    <YOUR_HOST>:2181
 
 Start Kyuubi
 ************
diff --git a/docs/deployment/high_availability_guide.md b/docs/deployment/high_availability_guide.md
index 51c878157..ca2ffa9fb 100644
--- a/docs/deployment/high_availability_guide.md
+++ b/docs/deployment/high_availability_guide.md
@@ -48,21 +48,21 @@ Using multiple Kyuubi service units with load balancing instead of a single unit
 
 When applying HA to Kyuubi deployment, we need to be aware of the below two thing basically,
 
-- `kyuubi.ha.zookeeper.quorum` - the external zookeeper cluster address for deploy a `k.i.`
-- `kyuubi.ha.zookeeper.namespace` - the root directory, a.k.a. the ServerSpace for deploy a `k.i.`
+- `kyuubi.ha.addresses` - the external zookeeper cluster address for deploy a `k.i.`
+- `kyuubi.ha.namespace` - the root directory, a.k.a. the ServerSpace for deploy a `k.i.`
 
 For more configurations, please see the HA section of [Introduction to the Kyuubi Configurations System](./settings.html#ha)
 
 ### Pseudo mode
 
-When `kyuubi.ha.zookeeper.quorum` is not configured, a `k.i.` will start an embedded zookeeper service and expose the address of itself there.
+When `kyuubi.ha.addresses` is not configured, a `k.i.` will start an embedded zookeeper service and expose the address of itself there.
 In this pseduo mode, the `k.i.` can be connected by clients through both raw ip address and zk quorum + namespace.
 But it doesn't have any availability to being highly available.
 
 ### Production mode
 
-For production deployment purpose, an external zookeeper cluster is required for `kyuubi.ha.zookeeper.quorum`.
-In this mode, multiple `k.i.`s can be registered to the same ServerSpace configured by `kyuubi.ha.zookeeper.namespace` and serve together.
+For production deployment purpose, an external zookeeper cluster is required for `kyuubi.ha.addresses`.
+In this mode, multiple `k.i.`s can be registered to the same ServerSpace configured by `kyuubi.ha.namespace` and serve together.
 
 ## Client-side Usage
 
diff --git a/docs/tools/kyuubi-ctl.rst b/docs/tools/kyuubi-ctl.rst
index 4a9308fed..407af4b56 100644
--- a/docs/tools/kyuubi-ctl.rst
+++ b/docs/tools/kyuubi-ctl.rst
@@ -112,7 +112,7 @@ Create server
 ***********
 Expose Kyuubi server instance to another domain.
 
-First read ``kyuubi.ha.zookeeper.namespace`` in ``conf/kyuubi-defaults.conf``, if there are server instances under this namespace, register them in the new namespace specified by the ``--namespace`` parameter.
+First read ``kyuubi.ha.namespace`` in ``conf/kyuubi-defaults.conf``, if there are server instances under this namespace, register them in the new namespace specified by the ``--namespace`` parameter.
 
 .. code-block:: bash
 
diff --git a/kyuubi-ha/src/main/scala/org/apache/kyuubi/ha/HighAvailabilityConf.scala b/kyuubi-ha/src/main/scala/org/apache/kyuubi/ha/HighAvailabilityConf.scala
index 626557008..e7a565284 100644
--- a/kyuubi-ha/src/main/scala/org/apache/kyuubi/ha/HighAvailabilityConf.scala
+++ b/kyuubi-ha/src/main/scala/org/apache/kyuubi/ha/HighAvailabilityConf.scala
@@ -68,7 +68,7 @@ object HighAvailabilityConf {
     "1.3.2")
   val HA_ZK_ACL_ENABLED: ConfigEntry[Boolean] =
     buildConf("kyuubi.ha.zookeeper.acl.enabled")
-      .doc("Set to true if the ZooKeeper ensemble is kerberized")
+      .doc("(deprecated) Set to true if the ZooKeeper ensemble is kerberized")
       .version("1.0.0")
       .booleanConf
       .createWithDefault(UserGroupInformation.isSecurityEnabled)
diff --git a/kyuubi-zookeeper/src/main/scala/org/apache/kyuubi/zookeeper/ZookeeperConf.scala b/kyuubi-zookeeper/src/main/scala/org/apache/kyuubi/zookeeper/ZookeeperConf.scala
index 9b0844e69..46c74ce6c 100644
--- a/kyuubi-zookeeper/src/main/scala/org/apache/kyuubi/zookeeper/ZookeeperConf.scala
+++ b/kyuubi-zookeeper/src/main/scala/org/apache/kyuubi/zookeeper/ZookeeperConf.scala
@@ -24,14 +24,14 @@ object ZookeeperConf {
 
   @deprecated("using kyuubi.zookeeper.embedded.client.port instead", since = "1.2.0")
   val EMBEDDED_ZK_PORT: ConfigEntry[Int] = buildConf("kyuubi.zookeeper.embedded.port")
-    .doc("The port of the embedded ZooKeeper server")
+    .doc("(deprecated) The port of the embedded ZooKeeper server")
     .version("1.0.0")
     .intConf
     .createWithDefault(2181)
 
   @deprecated("using kyuubi.zookeeper.embedded.data.dir instead", since = "1.2.0")
   val EMBEDDED_ZK_TEMP_DIR: ConfigEntry[String] = buildConf("kyuubi.zookeeper.embedded.directory")
-    .doc("The temporary directory for the embedded ZooKeeper server. " +
+    .doc("(deprecated) The temporary directory for the embedded ZooKeeper server. " +
       "If it is a relative path, it is resolved relative to KYUUBI_HOME. ")
     .version("1.0.0")
     .stringConf