From c06d4e1190d708294061ddb736fe673f34210cc0 Mon Sep 17 00:00:00 2001
From: Shanicky Chen <peng@risingwave-labs.com>
Date: Wed, 3 Jul 2024 18:30:25 +0800
Subject: [PATCH] Deprecate `vnode_mapping`, reserve field 5, update
 `MigrationPlan`

---
 proto/meta.proto | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/proto/meta.proto b/proto/meta.proto
index 100f5e45dff5f..bac2d6f7434f1 100644
--- a/proto/meta.proto
+++ b/proto/meta.proto
@@ -81,6 +81,8 @@ message TableFragments {
     uint32 fragment_type_mask = 2;
     FragmentDistributionType distribution_type = 3;
     repeated stream_plan.StreamActor actors = 4;
+
+    // NOTE: vnode_mapping is deprecated, we will generate the vnode_mapping by actors' bitmaps
     // Vnode mapping (which should be set in upstream dispatcher) of the fragment.
     // This field is always set to `Some`. For singleton, the parallel unit for all vnodes will be the same.
     reserved 5;
@@ -130,10 +132,12 @@ message ActorLocation {
 }
 
 message MigrationPlan {
+  // NOTE: parallel_unit_migration_plan is deprecated, using worker_slot_migration_plan instead
   // map<parallel_unit_id, parallel_unit>, the plan indicates that the actors will be migrated from old parallel unit to the new one.
   reserved 1;
   reserved "parallel_unit_migration_plan";
 
+  // map<src_worker_slot_id, dst_worker_slot_id>, the plan indicates that the actors will be migrated from old worker_slot to the new one.
   map<uint64, uint64> worker_slot_migration_plan = 2;
 }