apply review comments

This commit is contained in:
Tamo 2023-01-25 11:20:15 +01:00
parent bf94f89035
commit a858531574
4 changed files with 9 additions and 7 deletions

View File

@ -94,15 +94,17 @@ impl FileStore {
Ok(())
}
pub fn update_total_size(&self) -> Result<u64> {
/// Compute the size of all the updates contained in the file store.
pub fn compute_total_size(&self) -> Result<u64> {
let mut total = 0;
for uuid in self.all_uuids()? {
total += self.get_size(uuid?)?;
total += self.compute_size(uuid?).unwrap_or_default();
}
Ok(total)
}
pub fn get_size(&self, uuid: Uuid) -> Result<u64> {
/// Compute the size of one update
pub fn compute_size(&self, uuid: Uuid) -> Result<u64> {
Ok(self.get_update(uuid)?.metadata()?.len())
}

View File

@ -903,8 +903,8 @@ impl IndexScheduler {
}
/// The size on disk taken by all the updates files contained in the `IndexScheduler`, in bytes.
pub fn update_file_size(&self) -> Result<u64> {
Ok(self.file_store.update_total_size()?)
pub fn compute_update_file_size(&self) -> Result<u64> {
Ok(self.file_store.compute_total_size()?)
}
/// Delete a file from the index scheduler.

View File

@ -514,7 +514,7 @@ impl IndexScheduler {
.unwrap()
.any(|uuid| uuid.as_ref().unwrap() == &content_file),
"Could not find uuid `{content_file}` in the file_store. Available uuids are {:?}.",
self.file_store.all_uuids().unwrap().collect::<Result<Vec<_>>>().unwrap(),
self.file_store.all_uuids().unwrap().collect::<std::result::Result<Vec<_>, file_store::Error>>().unwrap(),
);
}
Status::Succeeded | Status::Failed | Status::Canceled => {

View File

@ -281,7 +281,7 @@ pub fn create_all_stats(
database_size += index_scheduler.size()?;
database_size += auth_controller.size()?;
database_size += index_scheduler.update_file_size()?;
database_size += index_scheduler.compute_update_file_size()?;
let stats = Stats { database_size, last_update: last_task, indexes };
Ok(stats)