Remove backup::schedule_periodic

CHANGELOG.md

@@ -68,6 +68,10 @@ Versioning](https://semver.org/spec/v2.0.0.html).
   - To ensure data integrity and avoid potential data loss, users currently
     utilizing review-database versions below 0.24.0 must migrate to version
     0.24.0 before proceeding with any further migrations.
+- The `backup::schedule_periodic` function has been permanently removed. This
+  decision has been made to streamline and focus on providing precise abstractions
+  for database operations. Users are advised to update their codebase accordingly
+  and leverage alternative methods for scheduling periodic backups.
 
 ### Fixed
 

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "review-database"
-version = "0.25.0-alpha.5"
+version = "0.25.0-alpha.6"
 edition = "2021"
 
 [dependencies]

src/backup.rs

@@ -4,8 +4,8 @@ use crate::Store;
 use anyhow::Result;
 use chrono::{DateTime, TimeZone, Utc};
 use rocksdb::backup::BackupEngineInfo;
-use std::{sync::Arc, time::Duration};
-use tokio::sync::{Notify, RwLock};
+use std::sync::Arc;
+use tokio::sync::RwLock;
 use tracing::{info, warn};
 
 #[allow(clippy::module_name_repetitions)]
@@ -25,37 +25,6 @@ impl From<BackupEngineInfo> for BackupInfo {
     }
 }
 
-/// Schedules periodic database backups.
-#[allow(clippy::module_name_repetitions)]
-pub async fn schedule_periodic(
-    store: Arc<RwLock<Store>>,
-    schedule: (Duration, Duration),
-    backups_to_keep: u32,
-    stop: Arc<Notify>,
-) {
-    use tokio::time::{sleep, Instant};
-
-    let (init, duration) = schedule;
-    let sleep = sleep(init);
-    tokio::pin!(sleep);
-
-    loop {
-        tokio::select! {
-            () = &mut sleep => {
-                sleep.as_mut().reset(Instant::now() + duration);
-                let _res = create(&store, false, backups_to_keep);
-            }
-            () = stop.notified() => {
-                info!("creating a database backup before shutdown");
-                let _res = create(&store, false, backups_to_keep);
-                stop.notify_one();
-                return;
-            }
-
-        }
-    }
-}
-
 /// Creates a new database backup, keeping the specified number of backups.
 ///
 /// # Errors

src/migration.rs

@@ -32,7 +32,7 @@ use tracing::info;
 /// // the database format won't be changed in the future alpha or beta versions.
 /// const COMPATIBLE_VERSION: &str = ">=0.5.0-alpha.2,<=0.5.0-alpha.4";
 /// ```
-const COMPATIBLE_VERSION_REQ: &str = ">=0.24.0,<=0.25.0-alpha.5";
+const COMPATIBLE_VERSION_REQ: &str = ">=0.24.0,<=0.25.0-alpha.6";
 
 /// Migrates data exists in `PostgresQL` to Rocksdb if necessary.
 ///