Closed HuSharp closed 1 week ago
@okJiang PTAL, thx!
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 77.28%. Comparing base (
cb28ca8
) to head (9f8de24
).:exclamation: Current head 9f8de24 differs from pull request most recent head 6f56dbc
Please upload reports for the commit 6f56dbc to get more accurate results.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: JmPotato, rleungx
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@HuSharp: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
pull-integration-realcluster-test | 6f56dbcab4e1c96e1c6ebbc5014eb4996e7e5b2c | link | false | /test pull-integration-realcluster-test |
Full PR test history. Your PR dashboard.
What problem does this PR solve?
Issue Number: Ref #4399
What is changed and how does it work?
issue relies on https://github.com/tikv/pd/pull/8301/commits/76b49327ff7842027631642ad1d3746c5b14a02a#diff-5cf563058fdae91ee59093d5721fde21b45888669570a5fdfd54bbe502ce3e03L93-L95
We have this logic for
clone
https://github.com/tikv/pd/blob/13174b5d4cab4d958f0b4ea2718ea7bb74992bc7/client/http/types.go#L366-L373This is because startKey is json:"-" which means cannot be Unmarshal from json, We need to take care of
Clone
method. https://github.com/tikv/pd/blob/13174b5d4cab4d958f0b4ea2718ea7bb74992bc7/client/http/types.go#L346-L349Check List
Tests
Release note