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 <[email protected]>
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 <[email protected]>
Signed-off-by: liangbowen <[email protected]>
---
.../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)