Deuxfleurs/garage
44
56
Fork 74
Code Issues 100 Pull requests 14 Releases 30 Wiki Activity

WIP add content defined chunking #42

Closed
trinity-1686a wants to merge 42 commits from content-defined-chunking into master
pull from: content-defined-chunking
merge into: Deuxfleurs:master
Conversation 3 Commits 42 Files changed 60 +5 -2
1 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit fae5104a2c - Show all commits

7
src/table/sync.rs
View file

@ -433,9 +433,12 @@ where
// We don't request those items from them, they will send them. // We don't request those items from them, they will send them.
// We only bother with pushing items that differ // We only bother with pushing items that differ
} }
MerkleNode::Leaf(ik, _) => { MerkleNode::Leaf(ik, ivhash) => {
// Just send that item directly // Just send that item directly
if let Some(val) = self.data.store.get(ik)? { if let Some(val) = self.data.store.get(&ik[..])? {
if blake2sum(&val[..]) != ivhash {
warn!("Hashes differ between stored value and Merkle tree, key: {:?} (if your server is very busy, don't worry, this happens when the Merkle tree can't be updated fast enough)", ik);
}
todo_items.push(val.to_vec()); todo_items.push(val.to_vec());
} }
} }