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 2023/04/24 01:47:52 UTC

[kyuubi] branch branch-1.7 updated: [KYUUBI #4753] [KYUUBI 4752][Improvement] KyuubiConf.unset should not log deprecation warning

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

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


The following commit(s) were added to refs/heads/branch-1.7 by this push:
     new a8256b7f9 [KYUUBI #4753] [KYUUBI 4752][Improvement] KyuubiConf.unset should not log deprecation warning
a8256b7f9 is described below

commit a8256b7f9125bc1e3e46eb2a9c7e45245ff8a185
Author: senmiaoliu <se...@trip.com>
AuthorDate: Mon Apr 24 09:47:15 2023 +0800

    [KYUUBI #4753] [KYUUBI 4752][Improvement] KyuubiConf.unset should not log deprecation warning
    
    ### _Why are the changes needed?_
    
    close #4752
    
    ### _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
    
    - [ ] [Run test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request
    
    Closes #4753 from lsm1/features/kyuubi_4752.
    
    Closes #4753
    
    8527a2bca [senmiaoliu] remove log deprecation warning in unset
    
    Authored-by: senmiaoliu <se...@trip.com>
    Signed-off-by: Cheng Pan <ch...@apache.org>
    (cherry picked from commit 6ba8b99151adeee2fbdc7127a1ec1b0a087d4787)
    Signed-off-by: Cheng Pan <ch...@apache.org>
---
 kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala | 1 -
 1 file changed, 1 deletion(-)

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 5301066cf..8b7fdeebc 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
@@ -103,7 +103,6 @@ case class KyuubiConf(loadSysDefault: Boolean = true) extends Logging {
 
   /** unset a parameter from the configuration */
   def unset(key: String): KyuubiConf = {
-    logDeprecationWarning(key)
     settings.remove(key)
     this
   }