Skip to content

Commit

Permalink
enhance: [2.5] Use mvcc timestamp as guarantee ts if set (#38980) (#3…
Browse files Browse the repository at this point in the history
…9019)

Cherry pick from master
pr: #38980
When MvccTimestamp is set, it could be used as guarantee timestamp
directly instead of new ts allocated by scheduler reducing the waiting
time when delegator has tsafe lag

Signed-off-by: Congqi Xia <[email protected]>
  • Loading branch information
congqixia authored Jan 7, 2025
1 parent 6f1febe commit 4b62dfd
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions internal/proxy/task_query.go
Original file line number Diff line number Diff line change
Expand Up @@ -570,6 +570,7 @@ func (t *queryTask) queryShard(ctx context.Context, nodeID int64, qn types.Query
retrieveReq.GetBase().TargetID = nodeID
if needOverrideMvcc && mvccTs > 0 {
retrieveReq.MvccTimestamp = mvccTs
retrieveReq.GuaranteeTimestamp = mvccTs
}

req := &querypb.QueryRequest{
Expand Down

0 comments on commit 4b62dfd

Please sign in to comment.