Skip to content

Commit

Permalink
kvserverpb: update comment on RER.Timestamp
Browse files Browse the repository at this point in the history
It was lying about helping with lease protection.

Release note: None
  • Loading branch information
andreimatei committed Apr 2, 2021
1 parent df928e4 commit 832bc80
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 21 deletions.
36 changes: 18 additions & 18 deletions pkg/kv/kvserver/kvserverpb/proposer_kv.pb.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 3 additions & 3 deletions pkg/kv/kvserver/kvserverpb/proposer_kv.proto
Original file line number Diff line number Diff line change
Expand Up @@ -125,9 +125,9 @@ message ReplicatedEvalResult {
Merge merge = 4;
ComputeChecksum compute_checksum = 21;
bool is_lease_request = 6;
// Duplicates BatchRequest.Timestamp for proposer-evaluated KV. Used
// to verify the validity of the command (for lease coverage and GC
// threshold).
// The BatchRequest.Timestamp of the request that produced this command. Used
// to verify the validity of the command against the GC threshold and to
// update the followers' clocks.
util.hlc.Timestamp timestamp = 8 [(gogoproto.nullable) = false];
// The stats delta corresponding to the data in this WriteBatch. On
// a split, contains only the contributions to the left-hand side.
Expand Down

0 comments on commit 832bc80

Please sign in to comment.