[KYUUBI #6212] Added audit handler shutdown to the shutdown hook
# 🔍 Description
This pull request fixes #6212
When Kyuubi cleans up Ranger related threads like PolicyRefresher, it should also shutdown the audit threads that include SolrZkClient. Otherwise Spark Driver keeps on running since SolrZkClient is a non-daemon thread. Added the cleanup as part of the shutdown hook that Kyuubi registers.
## Types of changes 🔖
- [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 ⚰️
#### Behavior With This Pull Request 🎉
#### 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 #6233 from amanraj2520/auditShutdown.
Closes #6212
e663d466c [amanraj2520] Refactored code
ed293a9a4 [amanraj2520] Removed unused import
95a6814ad [amanraj2520] Added audit handler shutdown to the shutdown hook
Authored-by: amanraj2520 <rajaman@microsoft.com>
Signed-off-by: Cheng Pan <chengpan@apache.org>
(cherry picked from commit 35d4b5f0c7)
Signed-off-by: Cheng Pan <chengpan@apache.org>
This commit is contained in:
parent
0f9f6cb56f
commit
de505c3d96
@ -78,6 +78,7 @@ object SparkRangerAdminPlugin extends RangerBasePlugin("spark", "sparkSql")
|
||||
if (plugin != null) {
|
||||
LOG.info(s"clean up ranger plugin, appId: ${plugin.getAppId}")
|
||||
plugin.cleanup()
|
||||
plugin.getAuditProviderFactory.shutdown()
|
||||
}
|
||||
},
|
||||
Integer.MAX_VALUE)
|
||||
|
||||
Loading…
Reference in New Issue
Block a user