block/manager.rs: Prioritize raw blocks when no compression configured #566
No reviewers
Labels
No Label
AdminAPI
Bug
Check AWS
CI
Correctness
Critical
Documentation
Ideas
Improvement
Low priority
Newcomer
Performance
S3 Compatibility
Testing
Usability
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: Deuxfleurs/garage#566
Loading…
Reference in New Issue
No description provided.
Delete Branch "jpds/garage:skip-compressed-blocks-scrub-no-compression"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
During a node scrub - I noticed that garage was making a large number of syscalls for non-existent
.zst
files.This adjusts
is_block_compressed()
to check if compression is enabled in the node configuration, and if not - it instead checks for the raw blocks first.0961b555e3
toc9f56ba97c
c9f56ba97c
to94470d5668
Sorry, ignore my previous comment, I didn't understand correctly.
@ -604,2 +609,4 @@
}
if fs::metadata(&path).await.is_ok() {
return Ok(true);
The true/false values returned also need to be swapped in the case we are checking the non-compressed file first.
94470d5668
to9899801472
9899801472
to9c788059e2
LGTM