-
Notifications
You must be signed in to change notification settings - Fork 4.1k
kv: include lease start time in raft command of lease transfer #62569
Copy link
Copy link
Closed
Labels
A-kv-distributionRelating to rebalancing and leasing.Relating to rebalancing and leasing.C-bugCode not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.GA-blocker
Description
See #60929 (comment).
As of c44b357, it is possible for the raft command of a lease transfer to carry a lower timestamp than the new lease itself. This can lead to situations where the new leaseholder has a clock that trails its new lease start time. This seems bad and we should fix it.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-kv-distributionRelating to rebalancing and leasing.Relating to rebalancing and leasing.C-bugCode not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.GA-blocker