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.

Sign up for GitHub

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

Jump to bottom

Remove InactiveRegionManager #2636

Closed
3 tasks done
WenyXu opened this issue Oct 23, 2023 · 0 comments · Fixed by #2729
Closed
3 tasks done

Remove InactiveRegionManager #2636

WenyXu opened this issue Oct 23, 2023 · 0 comments · Fixed by #2729
Assignees
@WenyXu
Labels
C-enhancement Category Enhancements
Milestone
v0.5

Comments

@WenyXu
Copy link
Member

WenyXu commented Oct 23, 2023 •
edited by fengjiachun
Loading

What type of enhancement is this?

Refactor

What does the enhancement do?

After we enable the LeaderCacheKvStore, we can remove the InactiveRegionManager and query the cache instead.

Steps:

Implementation challenges

No response
@WenyXu WenyXu added the C-enhancement Category Enhancements label Oct 23, 2023
@WenyXu WenyXu added this to the v0.5 milestone Oct 23, 2023
@WenyXu WenyXu self-assigned this Oct 24, 2023
@WenyXu WenyXu mentioned this issue Oct 24, 2023
Merged
2 tasks
@WenyXu WenyXu mentioned this issue Nov 10, 2023
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@WenyXu WenyXu

Labels
C-enhancement Category Enhancements
Projects
None yet
Milestone
v0.5
Development

Successfully merging a pull request may close this issue.

refactor: replace InactiveRegionManager with RegionLeaseKeeper WenyXu/greptimedb
1 participant
@WenyXu