refactor(aggregator): use const for vacuum minimum interval and rename constant

Author: dlachaume

mithril-aggregator/src/commands/serve_command.rs

@@ -16,6 +16,8 @@ use mithril_metric::MetricsServer;
 
 use crate::{dependency_injection::DependenciesBuilder, tools::VacuumTracker, Configuration};
 
+const VACUUM_MINIMUM_INTERVAL: TimeDelta = TimeDelta::weeks(1);
+
 /// Server runtime mode
 #[derive(Parser, Debug, Clone)]
 pub struct ServeCommand {
@@ -180,7 +182,7 @@ impl ServeCommand {
         self.perform_database_vacuum_if_needed(
             &config.data_stores_directory,
             &mut dependencies_builder,
-            TimeDelta::weeks(1),
+            VACUUM_MINIMUM_INTERVAL,
             &root_logger,
         )
         .await?;

mithril-aggregator/src/tools/vacuum_tracker.rs

@@ -9,7 +9,7 @@ use slog::{debug, info, Logger};
 
 use mithril_common::StdResult;
 
-const LAST_VACUUM_FILENAME: &str = "last_vacuum_time";
+const LAST_VACUUM_TIME_FILENAME: &str = "last_vacuum_time";
 
 type LastVacuumTime = DateTime<Utc>;
 
@@ -24,7 +24,7 @@ pub struct VacuumTracker {
 impl VacuumTracker {
     /// Create a new [VacuumTracker] for the given store directory
     pub fn new(store_dir: &Path, interval: TimeDelta, logger: Logger) -> Self {
-        let last_vacuum_file = store_dir.join(LAST_VACUUM_FILENAME);
+        let last_vacuum_file = store_dir.join(LAST_VACUUM_TIME_FILENAME);
 
         Self {
             tracker_file: last_vacuum_file,