Open terry1purcell opened 3 weeks ago
Hi @terry1purcell. Thanks for your PR.
I'm waiting for a pingcap member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
/ok-to-test
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 74.5481%. Comparing base (
8483dc5
) to head (132878c
).
@ZL91: adding LGTM is restricted to approvers and reviewers in OWNERS files.
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ZL91 Once this PR has been reviewed and has the lgtm label, please assign elsa0520 for approval. For more information see the Code Review Process.
The full list of commands accepted by this bot can be found here.
What problem does this PR solve?
Issue Number: close #53111
Problem Summary:
What changed and how does it work?
PR includes 3 changes to reduce unnecessary TopN values:
Check List
Tests
Side effects
Documentation
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.