kvserver: update allocator sync on errors in AdminTransferLease

Previously, the replicate queue could skip updating allocator sync when
rlm.AdminTransferLease failed, due to early returns on error. This commit
ensures allocator sync is always updated before exiting the function, regardless
of success or failure.

Author: wenyihu6

pkg/kv/kvserver/replicate_queue.go

@@ -1040,11 +1040,13 @@ func (rq *replicateQueue) TransferLease(
 		source,
 		target,
 	)
-	if err := rlm.AdminTransferLease(ctx, target.StoreID, false /* bypassSafetyChecks */); err != nil {
+
+	err := rlm.AdminTransferLease(ctx, target.StoreID, false /* bypassSafetyChecks */)
+	rq.as.PostApply(changeID, err == nil /*success*/)
+
+	if err != nil {
 		return errors.Wrapf(err, "%s: unable to transfer lease to s%d", rlm, target)
 	}
-
-	rq.as.PostApply(changeID, true /*success*/)
 	return nil
 }