You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by bo...@apache.org on 2023/05/12 14:36:22 UTC

[kyuubi] branch master updated: [KYUUBI #4834] [MINOR] Reduce the scope of method references in Authz plugin cleanup shutdown hook

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

bowenliang 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 474f0972a [KYUUBI #4834] [MINOR] Reduce the scope of method references in Authz plugin cleanup shutdown hook
474f0972a is described below

commit 474f0972a443c62c3d77a9977d61b59a18a0f0a6
Author: wforget <64...@qq.com>
AuthorDate: Fri May 12 22:36:10 2023 +0800

    [KYUUBI #4834] [MINOR] Reduce the scope of method references in Authz plugin cleanup shutdown hook
    
    ### _Why are the changes needed?_
    
    Reduce the scope of method references
    
    ### _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 #4834 from wForget/minor.
    
    Closes #4834
    
    2061dc942 [wforget] [MINOR] Reduce the scope of method references
    
    Authored-by: wforget <64...@qq.com>
    Signed-off-by: liangbowen <li...@gf.com.cn>
---
 .../kyuubi/plugin/spark/authz/ranger/SparkRangerAdminPlugin.scala       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/ranger/SparkRangerAdminPlugin.scala b/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/ranger/SparkRangerAdminPlugin.scala
index 8332b27f0..25bfca96a 100644
--- a/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/ranger/SparkRangerAdminPlugin.scala
+++ b/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/ranger/SparkRangerAdminPlugin.scala
@@ -79,7 +79,7 @@ object SparkRangerAdminPlugin extends RangerBasePlugin("spark", "sparkSql")
       () => {
         if (plugin != null) {
           LOG.info(s"clean up ranger plugin, appId: ${plugin.getAppId}")
-          this.cleanup()
+          plugin.cleanup()
         }
       },
       Integer.MAX_VALUE)