fix(storage): reset partition_vnode_count after merge (#20426)

risingwavelabs · Feb 8, 2025 · 1578865 · 1578865
1 parent 37d96fd
commit 1578865
Showing 1 changed file with 14 additions and 0 deletions.
diff --git a/src/meta/src/hummock/manager/compaction/compaction_group_schedule.rs b/src/meta/src/hummock/manager/compaction/compaction_group_schedule.rs
@@ -293,6 +293,20 @@ impl HummockManager {
                 );
             }
 
+            // clear `partition_vnode_count` for the hybrid group
+            {
+                if let Err(err) = compaction_groups_txn.update_compaction_config(
+                    &[left_group_id],
+                    &[MutableConfig::SplitWeightByVnode(0)], // default
+                ) {
+                    tracing::error!(
+                        error = %err.as_report(),
+                        "failed to update compaction config for group-{}",
+                        left_group_id
+                    );
+                }
+            }
+
             new_version_delta.pre_apply();
 
             // remove right_group_id