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

Improve the performance of handling region heartbeats #5648

Closed
14 of 16 tasks
rleungx opened this issue Oct 28, 2022 · 1 comment
Closed
14 of 16 tasks

Improve the performance of handling region heartbeats #5648

rleungx opened this issue Oct 28, 2022 · 1 comment
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@rleungx
Copy link
Member

rleungx commented Oct 28, 2022 •
edited
Loading

Enhancement Task

A large cluster, whose number of regions is more than 1M, could have the performance bottleneck of region heartbeat. The issue is used to track the progress of improving the heartbeat.

Here are some improvements we might do in the future:

Something we have done:
@rleungx rleungx added the type/enhancement The issue or PR belongs to an enhancement. label Oct 28, 2022
This was referenced Oct 28, 2022
Open
Merged
@JmPotato JmPotato mentioned this issue Nov 10, 2022
Merged
ti-chi-bot pushed a commit that referenced this issue Nov 10, 2022
@JmPotato
cluster: eliminate some duplicated code in processRegionHeartbeat (#5693
80801a2 
)

ref #5648

Eliminate some duplicated code in `processRegionHeartbeat`.

Signed-off-by: JmPotato <[email protected]>
ti-chi-bot pushed a commit that referenced this issue Nov 15, 2022
@rleungx
*: move update store to another place (#5652)
0e0f544 
ref #5648

Signed-off-by: Ryan Leung <[email protected]>
@rleungx rleungx pinned this issue Nov 16, 2022
@rleungx rleungx mentioned this issue Nov 17, 2022
Merged
@nolouch nolouch mentioned this issue Nov 18, 2022
Closed
ti-chi-bot added a commit that referenced this issue Nov 30, 2022
@rleungx @ti-chi-bot
pkg: support generic for btree (#5608)
5d34111 
ref #5606, ref #5648

Signed-off-by: Ryan Leung <[email protected]>

Co-authored-by: Ti Chi Robot <[email protected]>
This was referenced Nov 30, 2022
Merged
Merged
ti-chi-bot added a commit that referenced this issue Nov 30, 2022
@rleungx @ti-chi-bot
core: separate region lock (#5720)
58fde49 
ref #5586, ref #5648, ref #5712

Signed-off-by: Ryan Leung <[email protected]>

Co-authored-by: Ti Chi Robot <[email protected]>
@rleungx rleungx mentioned this issue Dec 2, 2022
Merged
ti-chi-bot pushed a commit that referenced this issue Dec 5, 2022
@rleungx
core: reorder the check (#5653)
4180772 
ref #5648

Signed-off-by: Ryan Leung <[email protected]>
ti-chi-bot added a commit that referenced this issue Dec 8, 2022
@rleungx @ti-chi-bot
*: reduce get overlap operations (#5615)
1b363d8 
ref #5606, ref #5648

Signed-off-by: Ryan Leung <[email protected]>

Co-authored-by: Ti Chi Robot <[email protected]>
@rleungx rleungx mentioned this issue Dec 8, 2022
Merged
ti-chi-bot pushed a commit that referenced this issue Dec 8, 2022
@rleungx
*: fix lock in CheckAndPutRegion (#5768)
8c5c666 
ref #5615, ref #5648

Signed-off-by: Ryan Leung <[email protected]>
@rleungx rleungx unpinned this issue Jan 4, 2023
@rleungx
Copy link
Member Author

rleungx commented Mar 3, 2023 •
edited
Loading

Close it for now since most of the jobs are finished.

@rleungx rleungx closed this as completed Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rleungx