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

Debouncing reschedule. #4416

Merged
merged 4 commits into from
Jan 23, 2024
Merged

Debouncing reschedule. #4416

merged 4 commits into from
Jan 23, 2024

Conversation

fulmicoton
Copy link
Contributor

After this change, the creation of a new plan will happen at most once by window of 5s. Outside of cooldown period, a (relevant) event triggers a RebuildPlan instant.

Related to #4407

@fulmicoton fulmicoton force-pushed the issue/4407-debounce-reschedule branch 3 times, most recently from 6314bf4 to 35bca11 Compare January 18, 2024 13:00
@fulmicoton fulmicoton marked this pull request as ready for review January 18, 2024 13:00
@fulmicoton fulmicoton force-pushed the issue/4407-debounce-reschedule branch from 0ec0c7d to 7208edb Compare January 18, 2024 13:46
quickwit/quickwit-control-plane/src/control_plane.rs Outdated Show resolved Hide resolved
quickwit/quickwit-control-plane/src/debouncer.rs Outdated Show resolved Hide resolved
quickwit/quickwit-serve/src/lib.rs Outdated Show resolved Hide resolved
quickwit/quickwit-control-plane/src/debouncer.rs Outdated Show resolved Hide resolved
@fulmicoton fulmicoton force-pushed the issue/4407-debounce-reschedule branch 3 times, most recently from 43693e5 to 37a1593 Compare January 19, 2024 07:47
After this change, the creation of a new plan will happen at most once
by window of 5s. Outside of cooldown period, a (relevant) event
triggers a RebuildPlan instant.

Limiting concurrency of the Metastore client using a tower Layer.

Closes #4407

Co-authored-by: Adrien Guillo <[email protected]>

cool_down -> cooldown
quickwit/quickwit-control-plane/src/debouncer.rs Outdated Show resolved Hide resolved
quickwit/quickwit-control-plane/src/debouncer.rs Outdated Show resolved Hide resolved
quickwit/quickwit-control-plane/src/debouncer.rs Outdated Show resolved Hide resolved
@fulmicoton fulmicoton enabled auto-merge (squash) January 23, 2024 01:42
@fulmicoton fulmicoton merged commit 0fa6348 into main Jan 23, 2024
4 checks passed
@fulmicoton fulmicoton deleted the issue/4407-debounce-reschedule branch January 23, 2024 01:42
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.

2 participants