forked from Deuxfleurs/garage
delete plain block when getting a compressed one
This commit is contained in:
parent
64818d3720
commit
5a52c4c7f3
1 changed files with 21 additions and 4 deletions
|
@ -57,6 +57,15 @@ pub enum BlockData {
|
||||||
Compressed(#[serde(with = "serde_bytes")] Vec<u8>),
|
Compressed(#[serde(with = "serde_bytes")] Vec<u8>),
|
||||||
}
|
}
|
||||||
|
|
||||||
|
impl BlockData {
|
||||||
|
pub fn is_compressed(&self) -> bool {
|
||||||
|
match self {
|
||||||
|
BlockData::Plain(_) => false,
|
||||||
|
BlockData::Compressed(_) => true,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
impl RpcMessage for Message {}
|
impl RpcMessage for Message {}
|
||||||
|
|
||||||
/// The block manager, handling block exchange between nodes, and block storage on local node
|
/// The block manager, handling block exchange between nodes, and block storage on local node
|
||||||
|
@ -155,9 +164,12 @@ impl BlockManager {
|
||||||
|
|
||||||
/// Write a block to disk
|
/// Write a block to disk
|
||||||
pub async fn write_block(&self, hash: &Hash, data: &BlockData) -> Result<Message, Error> {
|
pub async fn write_block(&self, hash: &Hash, data: &BlockData) -> Result<Message, Error> {
|
||||||
if self.is_block_compressed(hash).await.is_ok() {
|
let clean_plain = match self.is_block_compressed(hash).await {
|
||||||
return Ok(Message::Ok);
|
Ok(true) => return Ok(Message::Ok),
|
||||||
}
|
Ok(false) if !data.is_compressed() => return Ok(Message::Ok), // we have a plain block, and the provided block is not compressed either
|
||||||
|
Ok(false) => true,
|
||||||
|
Err(_) => false,
|
||||||
|
};
|
||||||
|
|
||||||
let mut path = self.block_dir(hash);
|
let mut path = self.block_dir(hash);
|
||||||
|
|
||||||
|
@ -177,11 +189,16 @@ impl BlockManager {
|
||||||
path.set_extension("zst_b2");
|
path.set_extension("zst_b2");
|
||||||
}
|
}
|
||||||
|
|
||||||
let mut f = fs::File::create(path).await?;
|
let mut f = fs::File::create(path.clone()).await?;
|
||||||
f.write_all(&buffer).await?;
|
f.write_all(&buffer).await?;
|
||||||
if let Some(checksum) = checksum {
|
if let Some(checksum) = checksum {
|
||||||
f.write_all(checksum.as_slice()).await?;
|
f.write_all(checksum.as_slice()).await?;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if clean_plain {
|
||||||
|
path.set_extension("");
|
||||||
|
fs::remove_file(path).await?;
|
||||||
|
}
|
||||||
drop(f);
|
drop(f);
|
||||||
|
|
||||||
Ok(Message::Ok)
|
Ok(Message::Ok)
|
||||||
|
|
Loading…
Reference in a new issue