[raft] fix a bug when we remove replica from rd (#8212)

luluz66 · web-flow · commit 4a352aeff1b7 · 2025-01-17T16:41:30.000-08:00
diff --git a/enterprise/server/raft/store/store.go b/enterprise/server/raft/store/store.go
@@ -2836,7 +2836,7 @@ func (s *Store) removeReplicaFromRangeDescriptor(ctx context.Context, rangeID, r
 	newDescriptor := proto.Clone(oldDescriptor).(*rfpb.RangeDescriptor)
 	for i, replica := range newDescriptor.Removed {
 		if replica.GetReplicaId() == replicaID {
-			newDescriptor.Removed = append(newDescriptor.Removed[:i], newDescriptor.Replicas[i+1:]...)
+			newDescriptor.Removed = append(newDescriptor.Removed[:i], newDescriptor.Removed[i+1:]...)
 			break
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -2836,7 +2836,7 @@ func (s *Store) removeReplicaFromRangeDescriptor(ctx context.Context, rangeID, r`
`2836`	`2836`	`newDescriptor := proto.Clone(oldDescriptor).(*rfpb.RangeDescriptor)`
`2837`	`2837`	`for i, replica := range newDescriptor.Removed {`
`2838`	`2838`	`if replica.GetReplicaId() == replicaID {`
`2839`		`- newDescriptor.Removed = append(newDescriptor.Removed[:i], newDescriptor.Replicas[i+1:]...)`
	`2839`	`+ newDescriptor.Removed = append(newDescriptor.Removed[:i], newDescriptor.Removed[i+1:]...)`
`2840`	`2840`	`break`
`2841`	`2841`	`}`
`2842`	`2842`	`}`