Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DNM]: for test #955

Open
wants to merge 18 commits into
base: tidb-6.5
Choose a base branch
from
Open

[DNM]: for test #955

wants to merge 18 commits into from

Conversation

cfzjywxk
Copy link
Contributor

Use tidb_kv_read_timeout as first kv request timeout (#919) (#948), run CI.

crazycs520 and others added 16 commits August 29, 2023 08:27
Signed-off-by: you06 <[email protected]>
add retry info to request source (#953) (#959)
…nt (#980) (#982)

* *: fix issue of configure kv timeout not work when disable batch client (#980)

* *: fix issue of configure kv timeout not work when disable batch client

Signed-off-by: crazycs520 <[email protected]>

* refine test

Signed-off-by: crazycs520 <[email protected]>

* address comment

Signed-off-by: crazycs520 <[email protected]>

---------

Signed-off-by: crazycs520 <[email protected]>

* fix test

Signed-off-by: crazycs520 <[email protected]>

---------

Signed-off-by: crazycs520 <[email protected]>
#983)

* *: configurable kv_read_timeout should not affect write request (#978)

---------

Signed-off-by: crazycs520 <[email protected]>

* add test

Signed-off-by: crazycs520 <[email protected]>

---------

Signed-off-by: crazycs520 <[email protected]>
)

* add retry limit for excluded backoff type to avoid infinite retry

Signed-off-by: cfzjywxk <[email protected]>

* lower the log level

Signed-off-by: cfzjywxk <[email protected]>

---------

Signed-off-by: cfzjywxk <[email protected]>
* fix the issue that health check may set liveness wrongly



* fix lint issue



* fix rawkv ut



* fix data race



* use getStore instead of accessing storeMu directly



* make TestAccessFollowerAfter1TiKVDown stable



* make TestBackoffErrorType stable



* address comments



---------

Signed-off-by: zyguan <[email protected]>
Co-authored-by: disksing <[email protected]>
…ore state and lead to unnecessary backoff (#1115) (#1337)

* Fix the issue that leader change with nea leader info cause invalidStore state and lead to unnecessary backoff (#1115)

* Fix the issue that leader change with nea leader info cause invalidStore
state and lead to unnecessary backoff

Signed-off-by: MyonKeminta <[email protected]>

* address comments

Signed-off-by: MyonKeminta <[email protected]>

---------

Signed-off-by: MyonKeminta <[email protected]>
Co-authored-by: MyonKeminta <[email protected]>

* Fix build on go1.18

Signed-off-by: MyonKeminta <[email protected]>

---------

Signed-off-by: MyonKeminta <[email protected]>
Co-authored-by: MyonKeminta <[email protected]>
* make experimental improvements to replica selector

Signed-off-by: zyguan <[email protected]>

* address the comment

Signed-off-by: zyguan <[email protected]>

* fix the test

Signed-off-by: zyguan <[email protected]>

---------

Signed-off-by: zyguan <[email protected]>
* improve region request log for diagnose

Signed-off-by: crazycs520 <[email protected]>

* remove useless filed

Signed-off-by: crazycs520 <[email protected]>

* use context logger as much as possible (#908)

Signed-off-by: crazycs520 <[email protected]>

* add leader peer id log

Signed-off-by: crazycs520 <[email protected]>

* *: fix panic in get cause error (#1344)

Signed-off-by: crazycs520 <[email protected]>

* address comment

Signed-off-by: crazycs520 <[email protected]>

---------

Signed-off-by: crazycs520 <[email protected]>
… when the store restarts with label changed (#1402)

* fix issue that store's liveness may incorrectly marked as unreachable when the store restarts with label changed

Signed-off-by: crazycs520 <[email protected]>

* refine

Signed-off-by: crazycs520 <[email protected]>

* remove unnecessary code

Signed-off-by: crazycs520 <[email protected]>

* refine log

Signed-off-by: crazycs520 <[email protected]>

* add log

Signed-off-by: crazycs520 <[email protected]>

* refine

Signed-off-by: crazycs520 <[email protected]>

* fix test

Signed-off-by: crazycs520 <[email protected]>

* refine log

Signed-off-by: crazycs520 <[email protected]>

* fix test

Signed-off-by: crazycs520 <[email protected]>

---------

Signed-off-by: crazycs520 <[email protected]>
@ti-chi-bot ti-chi-bot bot added the dco-signoff: yes Indicates the PR's author has signed the dco. label Jul 31, 2024
crazycs520 and others added 2 commits August 8, 2024 11:02
…1511)

* retry as stale reads

Signed-off-by: rishabh_mittal <[email protected]>

* build failure

Signed-off-by: rishabh_mittal <[email protected]>

* fixed test cases

Signed-off-by: rishabh_mittal <[email protected]>

* allow stale read if leader is not even tried

Signed-off-by: rishabh_mittal <[email protected]>

* fixed test cases

Signed-off-by: rishabh_mittal <[email protected]>

* disable test

Signed-off-by: rishabh_mittal <[email protected]>

* fixed test cases

Signed-off-by: rishabh_mittal <[email protected]>

---------

Signed-off-by: rishabh_mittal <[email protected]>
Copy link

ti-chi-bot bot commented Dec 19, 2024

[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 cfzjywxk, ensuring that each of them provides their approval before proceeding. For more information see the Code Review Process.

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

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 ti-chi-bot bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dco-signoff: yes Indicates the PR's author has signed the dco. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants