Skip to content

*: PR for test[DNM]#46694

Closed
crazycs520 wants to merge 17 commits intopingcap:release-6.5from
crazycs520:info-cache-variable-test
Closed

*: PR for test[DNM]#46694
crazycs520 wants to merge 17 commits intopingcap:release-6.5from
crazycs520:info-cache-variable-test

Conversation

@crazycs520
Copy link
Contributor

@crazycs520 crazycs520 commented Sep 5, 2023

What problem does this PR solve?

Issue Number: close #xxx

Problem Summary:

What is changed and how it works?

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Side effects

  • Performance regression: Consumes more CPU
  • Performance regression: Consumes more Memory
  • Breaking backward compatibility

Documentation

  • Affects user behaviors
  • Contains syntax changes
  • Contains variable changes
  • Contains experimental features
  • Changes MySQL compatibility

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.

None

crazycs520 and others added 14 commits September 1, 2023 21:54
@crazycs520 @hihihuhu @cfzjywxk
kv request: configurable KV Timeout (pingcap#45601) (pingcap#46274)
7055113 
* kv request: configurable KV Timeout (pingcap#45601)

close pingcap#45380

* update go.mod

Signed-off-by: crazycs520 <crazycs520@gmail.com>

* refine code

Signed-off-by: crazycs520 <crazycs520@gmail.com>

* make bazel_prepare

Signed-off-by: crazycs520 <crazycs520@gmail.com>

* update go.mod

Signed-off-by: crazycs520 <crazycs520@gmail.com>

* update go.mod

Signed-off-by: crazycs520 <crazycs520@gmail.com>

---------

Signed-off-by: crazycs520 <crazycs520@gmail.com>
Co-authored-by: Chen Ding <dingc05@gmail.com>
@you06 @cfzjywxk
session: set request source to type of the prepared stmt for execute …
c4a77fc 
…stmt (pingcap#46348) (pingcap#46362)

close pingcap#46349
@crazycs520 @cfzjywxk
fix test
69034ab 
Signed-off-by: crazycs520 <crazycs520@gmail.com>
@crazycs520 @cfzjywxk
fix test
876beda 
Signed-off-by: crazycs520 <crazycs520@gmail.com>
@cfzjywxk
update go.mod
af89760 
Signed-off-by: cfzjywxk <lsswxrxr@163.com>
@crazycs520 @cfzjywxk
make bazel_prepare
db4dac8 
Signed-off-by: crazycs520 <crazycs520@gmail.com>
@crazycs520 @cfzjywxk
make bazel_prepare
a936041 
Signed-off-by: crazycs520 <crazycs520@gmail.com>
@crazycs520 @cfzjywxk
tests: make test TestRuntimeHintsInEvolveTasks stable (pingcap#46152)
97750a2 
close pingcap#46124
@crazycs520 @cfzjywxk
fix test
cb36e18 
Signed-off-by: crazycs520 <crazycs520@gmail.com>
@crazycs520
metrics: refine metrics of infoschema (pingcap#46612)
f0f4dea 
ref pingcap#46524
@crazycs520
refien
2b0af39 
Signed-off-by: crazycs520 <crazycs520@gmail.com>
@crazycs520
refine
a5ec8b5 
Signed-off-by: crazycs520 <crazycs520@gmail.com>
@crazycs520
add global variable tidb_info_schema_cache_size to control infoschem…
3fe29f8 
…a cache size

    Signed-off-by: crazycs520 <crazycs520@gmail.com>
@crazycs520
refine
f7d8eee 
Signed-off-by: crazycs520 <crazycs520@gmail.com>
@crazycs520 crazycs520 requested a review from a team as a code owner September 5, 2023 15:08
@ti-chi-bot ti-chi-bot bot added release-note-none Denotes a PR that doesn't merit a release note. do-not-merge/cherry-pick-not-approved labels Sep 5, 2023
@ti-chi-bot
Copy link

ti-chi-bot bot commented Sep 5, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from crazycs520. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot
Copy link

ti-chi-bot bot commented Sep 5, 2023

This cherry pick PR is for a release branch and has not yet been approved by release team.
Adding the do-not-merge/cherry-pick-not-approved label.

To merge this cherry pick, it must first be approved by the collaborators.

AFTER it has been approved by collaborators, please ping the release team in a comment to request a cherry pick review.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@crazycs520 crazycs520 added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. and removed do-not-merge/cherry-pick-not-approved labels Sep 5, 2023
@ti-chi-bot ti-chi-bot bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. do-not-merge/cherry-pick-not-approved labels Sep 5, 2023
@ti-chi-bot
Copy link

ti-chi-bot bot commented Sep 5, 2023

This cherry pick PR is for a release branch and has not yet been approved by release team.
Adding the do-not-merge/cherry-pick-not-approved label.

To merge this cherry pick, it must first be approved by the collaborators.

AFTER it has been approved by collaborators, please ping the release team in a comment to request a cherry pick review.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@crazycs520
Copy link
Contributor Author

crazycs520 commented Sep 5, 2023

/ok-to-test

@ti-chi-bot ti-chi-bot bot added the ok-to-test Indicates a PR is ready to be tested. label Sep 5, 2023
@tiprow
Copy link

tiprow bot commented Sep 5, 2023

Hi @crazycs520. 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.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@crazycs520 crazycs520 mentioned this pull request Sep 5, 2023
Merged
12 tasks
@crazycs520 crazycs520 force-pushed the info-cache-variable-test branch from fa71a6a to 06adc22 Compare September 6, 2023 14:23
@crazycs520
go mod tidy
b704d9a 
Signed-off-by: crazycs520 <crazycs520@gmail.com>
@ti-chi-bot
Copy link

ti-chi-bot bot commented Sep 7, 2023

@crazycs520: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
idc-jenkins-ci-tidb/unit-test b704d9a link true /test unit-test
idc-jenkins-ci-tidb/build b704d9a link true /test build
idc-jenkins-ci-tidb/check_dev b704d9a link true /test check-dev
idc-jenkins-ci-tidb/check_dev_2 b704d9a link true /test check-dev2

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@cfzjywxk
Copy link
Contributor

cfzjywxk commented Sep 7, 2023

@crazycs520 Please add description and reference related issue if it's a pick PR by hand.

@ti-chi-bot ti-chi-bot bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 8, 2023
@ti-chi-bot
Copy link

ti-chi-bot bot commented Sep 8, 2023

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@crazycs520 crazycs520 closed this Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

do-not-merge/cherry-pick-not-approved do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. ok-to-test Indicates a PR is ready to be tested. release-note-none Denotes a PR that doesn't merit a release note. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@crazycs520 @cfzjywxk @you06