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

avoid cache avalanche #181

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

xpineal
Copy link

@xpineal xpineal commented Feb 10, 2022

At the beginning of client start, there is no table region info in cache.
If there are many go routine to access a table at same time, they will miss cache and send request package to get table region info.
In this case, too many duplicated requests should be avoid.

At the beginning of client start, there is no table region info in cache.
If there are many go routine to access a table at same time, they will miss cache and send request package to get table region info.
In this case, too many duplicated requests should be avoid.
@dethi
Copy link
Collaborator

dethi commented Feb 10, 2022

This looks similar to #164, not in the implementation but in what it tries to do.

@tsuna
Copy link
Owner

tsuna commented Dec 12, 2023

Discussed this with @dethi today and after looking at data on our clusters with @aaronbee we decided to add an option to the client to perform a cache warm up during initialization by preloading all entries from meta for tables of interest.

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

Successfully merging this pull request may close these issues.

None yet

3 participants