fix(cubestore): Wrong calculation of default compaction trigger size (too often compaction) (#7960)

Author: ovr

rust/cubestore/cubestore/src/config/mod.rs

@@ -1083,7 +1083,12 @@ where
 pub fn env_parse_size(name: &str, default: usize, max: Option<usize>, min: Option<usize>) -> usize {
     let v = match env::var(name).ok() {
         None => {
-            return default;
+            if cfg!(debug_assertions) {
+                // It's needed to check that default values are correct
+                default.to_string()
+            } else {
+                return default;
+            }
         }
         Some(v) => v,
     };
@@ -1135,17 +1140,16 @@ where
 
 impl Config {
     fn calculate_cache_compaction_trigger_size(cache_max_size: usize) -> usize {
-        match cache_max_size >> 20 {
-            // TODO: Enable this limits after moving to separate CF for cache
-            // d if d < 32 => 32 * 9,
-            // d if d < 64 => 64 * 8,
-            // d if d < 128 => 128 * 7,
-            // d if d < 256 => 256 * 6,
-            d if d < 512 => 512 * 5,
+        let trigger_size = match cache_max_size >> 20 {
+            d if d < 32 => (640 << 20),
+            d if d < 64 => (1280 << 20),
+            d if d < 512 => cache_max_size * 5,
             d if d < 1024 => cache_max_size * 4,
             d if d < 4096 => cache_max_size * 3,
             _ => cache_max_size * 2,
-        }
+        };
+
+        std::cmp::max(trigger_size, 512 << 20)
     }
 
     pub fn default() -> Config {