Closed evenyag closed 1 week ago
A new loop has been added to the checkpoint.rs
file in the manifest/tests
directory. This loop ensures that the process waits for a checkpoint to complete before proceeding with further operations, enhancing the reliability and correctness of the tests.
Files | Change Summary |
---|---|
src/mito2/src/manifest/tests/checkpoint.rs |
Added a loop to wait for checkpoint completion |
In the code where checkpoints lie, Patience waits, and tests comply. A loop to watch, a pause to see, Ensures our code runs error-free. 🐰 Through waits and checks, we stride with cheer, For flawless tests, we hold dear dear. 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.76%. Comparing base (
cdd4baf
) to head (3005cbe
). Report is 1 commits behind head on main.
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
https://github.com/GreptimeTeam/greptimedb/issues/4191
What's changed and what's your intention?
The checkpoint is done in the background so the checkpoint test may not corrupt the latest checkpoint. This PR waits until the checkpoint is finished.
Checklist
Summary by CodeRabbit