Skip to content

Commit ff8481a

Browse files
tbgsumeerbhola
authored andcommitted
mmaintegration: some assertion gardening
1 parent a8b8441 commit ff8481a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

pkg/kv/kvserver/mmaintegration/mma_conversion.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@ func convertReplicaChangeToMMA(
158158
}
159159
if lhBeingRemoved && !pickedLeaseholder {
160160
return mmaprototype.PendingRangeChange{},
161-
errors.Errorf("leaseholder being removed but no new leaseholder picked from %v", changes)
161+
errors.AssertionFailedf("leaseholder being removed but no new leaseholder picked from %v", changes)
162162
}
163163
for target, ch := range changeMap {
164164
var replicaChange mmaprototype.ReplicaChange
@@ -170,7 +170,7 @@ func convertReplicaChangeToMMA(
170170
case mmaprototype.ChangeReplica:
171171
replicaChange = mmaprototype.MakeReplicaTypeChange(desc.RangeID, rLoad, ch.prev, ch.next, target)
172172
default:
173-
panic("unimplemented change type")
173+
panic(errors.AssertionFailedf("unimplemented change type"))
174174
}
175175
replicaChanges = append(replicaChanges, replicaChange)
176176
}

0 commit comments

Comments
 (0)