Remove redundant log message in backup

CHANGELOG.md

@@ -27,6 +27,9 @@ Versioning](https://semver.org/spec/v2.0.0.html).
   with other database table type definitions.
 - Changed the return type of `Store::account_policy_map` to `Table<AccountPlicy>`
   to enhance security by preventing direct exposure of internal structure.
+- Removed redundant log messages in the backup module. Errors are now reported
+  through return values only, allowing callers to handle and log errors
+  according to their needs.
 
 ### Removed
 

src/backup.rs

@@ -6,7 +6,6 @@
 use rocksdb::backup::BackupEngineInfo;
 use std::sync::Arc;
 use tokio::sync::RwLock;
-use tracing::{info, warn};
 
 #[allow(clippy::module_name_repetitions)]
 pub struct BackupInfo {
@@ -32,21 +31,8 @@
 /// Returns an error if backup fails.
 pub async fn create(store: &Arc<RwLock<Store>>, flush: bool, backups_to_keep: u32) -> Result<()> {
     // TODO: This function should be expanded to support PostgreSQL backups as well.
-    info!("backing up database...");
-    let res = {
-        let mut store = store.write().await;
-        store.backup(flush, backups_to_keep)
-    };
-    match res {
-        Ok(()) => {
-            info!("backing up database completed");
-            Ok(())
-        }
-        Err(e) => {
-            warn!("database backup failed: {:?}", e);
-            Err(e)
-        }
-    }
+    let mut store = store.write().await;
+    store.backup(flush, backups_to_keep)
 }
 
 /// Lists the backup information of the database.
@@ -56,23 +42,14 @@
 /// Returns an error if backup list fails to create
 pub async fn list(store: &Arc<RwLock<Store>>) -> Result<Vec<BackupInfo>> {
     // TODO: This function should be expanded to support PostgreSQL backups as well.
-    let res = {
+    let backup_list = {
         let store = store.read().await;
-        store.get_backup_info()
+        store.get_backup_info()?
     };
-    match res {
-        Ok(backup_list) => {
-            info!("generate database backup list");
-            Ok(backup_list
-                .into_iter()
-                .map(std::convert::Into::into)
-                .collect())
-        }
-        Err(e) => {
-            warn!("failed to generate backup list: {:?}", e);
-            Err(e)
-        }
-    }
+    Ok(backup_list
+        .into_iter()
+        .map(std::convert::Into::into)
+        .collect())
 }
 
 /// Restores the database from a backup with the specified ID.