Closed okJiang closed 2 weeks ago
Attention: Patch coverage is 47.22222%
with 19 lines
in your changes missing coverage. Please review.
Project coverage is 77.28%. Comparing base (
9dff6e6
) to head (b2b5ab6
). Report is 1 commits behind head on master.
/retest
/merge
@JmPotato: We have migrated to builtin LGTM
and approve
plugins for reviewing.
👉 Please use /approve
when you want approve this pull request.
The changes announcement: Proposal: Strengthen configuration change approval.
/approve
[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
/merge
@rleungx: We have migrated to builtin LGTM
and approve
plugins for reviewing.
👉 Please use /approve
when you want approve this pull request.
The changes announcement: Proposal: Strengthen configuration change approval.
What problem does this PR solve?
Issue Number: Ref https://github.com/tikv/pd/issues/1919
What is changed and how does it work?
Check List
Tests
Code changes
Side effects
Related changes
pingcap/docs
/pingcap/docs-cn
:pingcap/tiup
:Release note