Skip to content

Commit 4a0ad21

Browse files
committed
kvserver: pass replication target rq.TransferLease
Previously, rq.TransferLease was updated to take in replication target (node ID and store ID) instead of just a store ID, but the log line formatting wasn’t updated accordingly. This commit fixes the log formatting.
1 parent ac24cb9 commit 4a0ad21

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

pkg/kv/kvserver/replicate_queue.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1095,7 +1095,7 @@ func (rq *replicateQueue) TransferLease(
10951095
rangeUsageInfo allocator.RangeUsageInfo,
10961096
) error {
10971097
rq.metrics.TransferLeaseCount.Inc(1)
1098-
log.KvDistribution.Infof(ctx, "transferring lease to s%d", target)
1098+
log.KvDistribution.Infof(ctx, "transferring lease to %v", target)
10991099
// Inform allocator sync that the change has been applied which applies
11001100
// changes to store pool and inform mma.
11011101
changeID := rq.as.NonMMAPreTransferLease(
@@ -1109,7 +1109,7 @@ func (rq *replicateQueue) TransferLease(
11091109
rq.as.PostApply(changeID, err == nil /*success*/)
11101110

11111111
if err != nil {
1112-
return errors.Wrapf(err, "%s: unable to transfer lease to s%d", rlm, target)
1112+
return errors.Wrapf(err, "%s: unable to transfer lease to %v", rlm, target)
11131113
}
11141114
return nil
11151115
}

0 commit comments

Comments
 (0)