Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expire CredentialsRef in a proper time to reduce memory usage #1021

Closed
zhouyifan279 opened this issue Sep 3, 2021 · 2 comments
Closed

Expire CredentialsRef in a proper time to reduce memory usage #1021

zhouyifan279 opened this issue Sep 3, 2021 · 2 comments
Assignees
Milestone

Comments

@zhouyifan279
Copy link
Contributor

zhouyifan279 commented Sep 3, 2021

1. Describe the feature

This is the a sub-task under umbrella issue #913.

In sub-task #1007, <user, CredentialsRef> pairs are cached in a Map. Credentials are periodically renewed and will never be removed.
If a lot of inactive users' credentials are cached, both memory usage and credentials renewal load may become unacceptable.
So we should implement a logic to expire cached <user, CredentialsRef> pairs to erase this risk.

2. Motivation

3. Describe the solution

4. Additional context

@zhouyifan279 zhouyifan279 changed the title Evict Expire CredentialsRef in a proper time to reduce memory usage Sep 3, 2021
@lightning-L
Copy link
Contributor

Hi, I'd like to try to implement this.

@ulysses-you
Copy link
Contributor

@lightning-L assigned, thank you

@yaooqinn yaooqinn added this to the v1.6.0 milestone Mar 16, 2022
lightning-L added a commit to lightning-L/kyuubi that referenced this issue Mar 24, 2022
lightning-L added a commit to lightning-L/kyuubi that referenced this issue Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants