Closed AilinKid closed 5 days ago
Hi @AilinKid. Thanks for your PR.
PRs from untrusted users cannot be marked as trusted with /ok-to-test
in this repo meaning untrusted PR authors can never trigger tests themselves. Collaborators can still trigger tests on the PR using /test all
.
I understand the commands that are listed here.
/ok-to-test
Attention: Patch coverage is 90.63136%
with 46 lines
in your changes are missing coverage. Please review.
Project coverage is 74.6691%. Comparing base (
9657f06
) to head (ec3d365
). Report is 10 commits behind head on master.
/test unit-test
/test fast_test_tiprow
@AilinKid: The specified target(s) for /test
were not found.
The following commands are available to trigger required jobs:
/test build
/test check-dev
/test check-dev2
/test mysql-test
/test pull-br-integration-test
/test pull-integration-ddl-test
/test pull-lightning-integration-test
/test pull-mysql-client-test
/test unit-test
The following commands are available to trigger optional jobs:
/test canary-notify-when-compatibility-sections-changed
/test pingcap/tidb/canary_ghpr_unit_test
/test pull-common-test
/test pull-e2e-test
/test pull-integration-common-test
/test pull-integration-copr-test
/test pull-integration-jdbc-test
/test pull-integration-mysql-test
/test pull-integration-nodejs-test
/test pull-sqllogic-test
/test pull-tiflash-test
Use /test all
to run the following jobs that were automatically triggered:
pingcap/tidb/ghpr_build
pingcap/tidb/ghpr_check
pingcap/tidb/ghpr_check2
pingcap/tidb/ghpr_mysql_test
pingcap/tidb/ghpr_unit_test
pingcap/tidb/pull_integration_ddl_test
pingcap/tidb/pull_mysql_client_test
@AilinKid: The specified target(s) for /test
were not found.
The following commands are available to trigger required jobs:
/test fast_test_tiprow
/test tidb_parser_test
Use /test all
to run all jobs.
/test fast_test_tiprow
@AilinKid: The specified target(s) for /test
were not found.
The following commands are available to trigger required jobs:
/test build
/test check-dev
/test check-dev2
/test mysql-test
/test pull-br-integration-test
/test pull-integration-ddl-test
/test pull-lightning-integration-test
/test pull-mysql-client-test
/test unit-test
The following commands are available to trigger optional jobs:
/test canary-notify-when-compatibility-sections-changed
/test pingcap/tidb/canary_ghpr_unit_test
/test pull-common-test
/test pull-e2e-test
/test pull-integration-common-test
/test pull-integration-copr-test
/test pull-integration-jdbc-test
/test pull-integration-mysql-test
/test pull-integration-nodejs-test
/test pull-sqllogic-test
/test pull-tiflash-test
Use /test all
to run the following jobs that were automatically triggered:
pingcap/tidb/ghpr_build
pingcap/tidb/ghpr_check
pingcap/tidb/ghpr_check2
pingcap/tidb/ghpr_mysql_test
pingcap/tidb/ghpr_unit_test
pingcap/tidb/pull_integration_ddl_test
pingcap/tidb/pull_mysql_client_test
/test fast_test_tiprow
@AilinKid: The specified target(s) for /test
were not found.
The following commands are available to trigger required jobs:
/test build
/test check-dev
/test check-dev2
/test mysql-test
/test pull-br-integration-test
/test pull-integration-ddl-test
/test pull-lightning-integration-test
/test pull-mysql-client-test
/test unit-test
The following commands are available to trigger optional jobs:
/test canary-notify-when-compatibility-sections-changed
/test pingcap/tidb/canary_ghpr_unit_test
/test pull-common-test
/test pull-e2e-test
/test pull-integration-common-test
/test pull-integration-copr-test
/test pull-integration-jdbc-test
/test pull-integration-mysql-test
/test pull-integration-nodejs-test
/test pull-sqllogic-test
/test pull-tiflash-test
Use /test all
to run the following jobs that were automatically triggered:
pingcap/tidb/ghpr_build
pingcap/tidb/ghpr_check
pingcap/tidb/ghpr_check2
pingcap/tidb/ghpr_mysql_test
pingcap/tidb/ghpr_unit_test
pingcap/tidb/pull_integration_ddl_test
pingcap/tidb/pull_mysql_client_test
/test check-dev2
@AilinKid: The specified target(s) for /test
were not found.
The following commands are available to trigger required jobs:
/test fast_test_tiprow
/test tidb_parser_test
Use /test all
to run all jobs.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hawkingrei, winoros
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Timeline:
2024-05-17 04:03:54.829090499 +0000 UTC m=+1798788.586226070
: :ballot_box_with_check: agreed by hawkingrei.2024-05-22 10:44:42.977720271 +0000 UTC m=+2254836.734855844
: :ballot_box_with_check: agreed by winoros./test unit-test
@AilinKid: The specified target(s) for /test
were not found.
The following commands are available to trigger required jobs:
/test fast_test_tiprow
/test tidb_parser_test
Use /test all
to run all jobs.
What problem does this PR solve?
Issue Number: ref https://github.com/pingcap/tidb/issues/51664, ref https://github.com/pingcap/tidb/issues/52714
Problem Summary:
What changed and how does it work?
/base
is where is interface definition./operator
is where in inheritors defined, interface function should be implemented here./core
is where the main optimizing logic happens./operator
depended on/based
./core
depends on/operator
and/base
.Check List
Tests
Side effects
Documentation
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.