Fix stale read can not eliminate the learner read on TiFlash#10099
Fix stale read can not eliminate the learner read on TiFlash#10099ti-chi-bot[bot] merged 2 commits intomasterfrom
Conversation
|
Hi @dependabot[bot]. 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 Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions 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-sigs/prow repository. |
Bumps [contrib/tiflash-proxy](https://github.com/pingcap/tidb-engine-ext) from `1157849` to `54aa039`. - [Commits](pingcap/tidb-engine-ext@1157849...54aa039) --- updated-dependencies: - dependency-name: contrib/tiflash-proxy dependency-version: 54aa039ba3e9ca9fffe3cf6c344f2822b405b3e4 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
56c83cc to
4b64a4e
Compare
|
/ok-to-test |
1157849 to 54aa039|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: CalvinNeo, JaySon-Huang The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
[LGTM Timeline notifier]Timeline:
|
|
In response to a cherrypick label: failed to apply #10099 on top of branch "release-8.1": |
|
In response to a cherrypick label: failed to apply #10099 on top of branch "release-8.5": |
…#10099) (pingcap#10132) ref pingcap#4982, close pingcap#10046 Signed-off-by: CalvinNeo <calvinneo1995@gmail.com>
|
In response to a cherrypick label: failed to apply #10099 on top of branch "release-8.5": |
Issue Number: ref #4982, close #10046
Bumps contrib/tiflash-proxy from
1157849to54aa039.The leader_safe_ts, self_safe_ts are always 0 in the TiFlash side, making TiFlash can not eliminate the learner read on region without any updates. For detailed reasons why safe_ts failed to be updated, checkout the description on pingcap/tidb-engine-ext#429.
To obtain the fixed PR in proxy: pingcap/tidb-engine-ext#429
Commits
54aa039Merge tikv master near 9.0-beta.1(e5a3075936ddf05461c8e1a1398b513badf3f876) (...9a6839aFix safe ts not updated (#429)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Release note