You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by fe...@apache.org on 2022/07/02 15:38:32 UTC

[incubator-kyuubi] branch master updated: [KYUUBI #2993] Fix typo in KyuubiConf and mark more config entries server only

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 163e0f823 [KYUUBI #2993] Fix typo in KyuubiConf and mark more config entries server only
163e0f823 is described below

commit 163e0f8232959b3fd365785f5a5312d2457bd290
Author: Fei Wang <fw...@ebay.com>
AuthorDate: Sat Jul 2 23:38:25 2022 +0800

    [KYUUBI #2993] Fix typo in KyuubiConf and mark more config entries server only
    
    ### _Why are the changes needed?_
    
    Fix typo in KyuubiConf and mark more config entries server only.
    
    ### _How was this patch tested?_
    - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible
    
    - [ ] Add screenshots for manual tests if appropriate
    
    - [x] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request
    
    Closes #2993 from turboFei/fix_typo.
    
    Closes #2993
    
    002e911f [Fei Wang] add more
    53ef59a0 [Fei Wang] save
    d5a55683 [Fei Wang] add into server only
    fea3daa4 [Fei Wang] Fix typo
    
    Authored-by: Fei Wang <fw...@ebay.com>
    Signed-off-by: Fei Wang <fw...@ebay.com>
---
 .../main/scala/org/apache/kyuubi/config/KyuubiConf.scala  | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala b/kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala
index bcabc7b91..5f85b9879 100644
--- a/kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala
+++ b/kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala
@@ -142,7 +142,18 @@ case class KyuubiConf(loadSysDefault: Boolean = true) extends Logging {
     FRONTEND_MYSQL_BIND_HOST,
     FRONTEND_MYSQL_BIND_PORT,
     AUTHENTICATION_METHOD,
-    KINIT_INTERVAL)
+    KINIT_INTERVAL,
+    SERVER_KEYTAB,
+    SERVER_PRINCIPAL,
+    SERVER_SPNEGO_KEYTAB,
+    SERVER_SPNEGO_PRINCIPAL,
+    SERVER_EVENT_LOGGERS,
+    SERVER_EVENT_JSON_LOG_PATH,
+    SERVER_OPERATION_LOG_DIR_ROOT,
+    SERVER_NAME,
+    SERVER_LIMIT_CONNECTIONS_PER_IPADDRESS,
+    SERVER_LIMIT_CONNECTIONS_PER_USER_IPADDRESS,
+    SERVER_LIMIT_CONNECTIONS_PER_USER)
 
   def getUserDefaults(user: String): KyuubiConf = {
     val cloned = KyuubiConf(false)
@@ -181,7 +192,7 @@ object KyuubiConf {
   private def register(entry: ConfigEntry[_]): Unit = kyuubiConfEntries.synchronized {
     require(
       !kyuubiConfEntries.containsKey(entry.key),
-      s"Duplicate SQLConfigEntry. ${entry.key} has been registered")
+      s"Duplicate ConfigEntry. ${entry.key} has been registered")
     kyuubiConfEntries.put(entry.key, entry)
   }