fix(storage): auto_pick_type should cover Tombstone type (#12670)

risingwavelabs · Oct 8, 2023 · 1d04366 · 1d04366
1 parent 129ab28
commit 1d04366
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/meta/src/hummock/manager/mod.rs b/src/meta/src/hummock/manager/mod.rs
@@ -3016,9 +3016,13 @@ impl CompactionState {
             Some(compact_task::TaskType::SpaceReclaim)
         } else if guard.contains(&(group, compact_task::TaskType::Ttl)) {
             Some(compact_task::TaskType::Ttl)
+        } else if guard.contains(&(group, compact_task::TaskType::Tombstone)) {
+            Some(compact_task::TaskType::Tombstone)
         } else if guard.contains(&(group, compact_task::TaskType::Dynamic)) {
             Some(compact_task::TaskType::Dynamic)
         } else {
+            // Other types are not triggered by `try_sched_compaction`, so no need to be handled.
+            // TODO: Consider using match state to avoid missing newly added types
             None
         }
     }