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

checker: make patrol region sooner #7963

Closed
lhy1024 opened this issue Mar 21, 2024 · 0 comments · Fixed by #8094
Closed

checker: make patrol region sooner #7963

lhy1024 opened this issue Mar 21, 2024 · 0 comments · Fixed by #8094
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@lhy1024
Copy link
Contributor

lhy1024 commented Mar 21, 2024 •
edited
Loading

Enhancement Task

image

We are now using one thread to scan all the regions and when the number of regions is high, it will always have more time without scheduling generation.

To enhance efficiency, we might consider implementing concurrent scanning or establishing a cache to prioritize the scanning of previously encountered regions that can generate operators but the key range is limited because of limit.
@lhy1024 lhy1024 added the type/enhancement The issue or PR belongs to an enhancement. label Mar 21, 2024
@rleungx rleungx mentioned this issue Apr 15, 2024
Merged
ti-chi-bot bot added a commit that referenced this issue Apr 16, 2024
@rleungx @ti-chi-bot
schedule: enlarge waiting list cache size (#7559)
f5fd36f 
ref #7963

Signed-off-by: Ryan Leung <rleungx@gmail.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@lhy1024 lhy1024 mentioned this issue May 23, 2024
Merged
@lhy1024 lhy1024 mentioned this issue Jul 22, 2024
Merged
ti-chi-bot bot added a commit that referenced this issue Aug 1, 2024
@lhy1024 @ti-chi-bot
schedule: move the logic of patrol region from coordinator to checkers (
3ce31ef 
#8428)

ref #7963

Signed-off-by: lhy1024 <admin@liudos.us>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@lhy1024 lhy1024 mentioned this issue Aug 1, 2024
Merged
ti-chi-bot bot added a commit that referenced this issue Aug 6, 2024
@lhy1024 @ti-chi-bot
schedule: adjust patrol region scan limit according to the num of re…
5e0fdb8 
…gions (#8479)

ref #7963

Signed-off-by: lhy1024 <admin@liudos.us>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
ti-chi-bot bot added a commit that referenced this issue Oct 30, 2024
@lhy1024 @ti-chi-bot
schedule: support patrol region concurrency (#8094)
649393a 
close #7963

Signed-off-by: lhy1024 <admin@liudos.us>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
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

Successfully merging a pull request may close this issue.

schedule: support patrol region concurrency lhy1024/pd
1 participant
@lhy1024