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 doing a trim first thing on startup #2671

Merged
merged 3 commits into from
Feb 8, 2025

Conversation

muhamadazmy
Copy link
Contributor

@muhamadazmy muhamadazmy commented Feb 7, 2025

Open on behalf of @pcholakov to replace #2662

Shifts the first trim check to happen at the configured trim check interval plus a small jitter factor, to avoid warnings on cluster startup.
@pcholakov
Copy link
Contributor

This is it, perfect! Thanks for digging up the commits and re-creating this, @muhamadazmy :-) going to merge this now.

@pcholakov pcholakov merged commit ae7bd6a into restatedev:main Feb 8, 2025
49 of 50 checks passed
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