-
Notifications
You must be signed in to change notification settings - Fork 409
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
[WIP] Proactive flush of delta merge layer phase 1 #7558
Conversation
Signed-off-by: hehechen <awd123456sss@gmail.com>
[REVIEW NOTIFICATION] This pull request has not been approved. To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
72b2ae3
to
2f873bf
Compare
Signed-off-by: CalvinNeo <calvinneo1995@gmail.com>
Signed-off-by: CalvinNeo <calvinneo1995@gmail.com>
c38afe9
to
6c5986f
Compare
/run-all-tests |
/run-build-release comment=true |
1 similar comment
/run-build-release comment=true |
/run-integration-test |
/run-build-release |
download tiflash binary(linux amd64) at http://fileserver.pingcap.net/download/builds/pingcap/test/tiflash/03a3da3e6e2b60ab192550ceba870b5b527f082f/centos7/tiflash-linux-amd64.tar.gz |
/run-build-release |
0aa3e77
to
9c0e733
Compare
/run-build-release comment=true |
download tiflash binary(linux amd64) at http://fileserver.pingcap.net/download/builds/pingcap/test/tiflash/0aa3e77ef733cd8313b6fd6f0bb1f4dcb972fdb1/centos7/tiflash-linux-amd64.tar.gz |
/run-build-release comment=true |
download tiflash binary(linux amd64) at http://fileserver.pingcap.net/download/builds/pingcap/test/tiflash/49c7bc0ab72f3ba2f55c79740816bbc6d11d1845/centos7/tiflash-linux-amd64.tar.gz |
Signed-off-by: CalvinNeo <calvinneo1995@gmail.com>
49c7bc0
to
7db53fe
Compare
/run-build-release comment=true |
download tiflash binary(linux amd64) at http://fileserver.pingcap.net/download/builds/pingcap/test/tiflash/7db53fe9feb94d9820e044b90d4de6576ee10a30/centos7/tiflash-linux-amd64.tar.gz |
When foreground flusing region 8880, we need also flush reigon 554
When forground flushing 554, we need also flushing 8880
|
Signed-off-by: CalvinNeo <calvinneo1995@gmail.com>
/run-build-release comment=true |
download tiflash binary(linux amd64) at http://fileserver.pingcap.net/download/builds/pingcap/test/tiflash/2dfaab1604c5bf52590ce76da3e027f70655d5cd/centos7/tiflash-linux-amd64.tar.gz |
Signed-off-by: CalvinNeo <calvinneo1995@gmail.com>
Signed-off-by: CalvinNeo <calvinneo1995@gmail.com>
Signed-off-by: CalvinNeo <calvinneo1995@gmail.com>
c173db5
to
6aa3dc6
Compare
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 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 |
/run-build-release comment=true |
1 similar comment
/run-build-release comment=true |
ec97a43
to
6b7b256
Compare
/run-build-release comment=true |
download tiflash binary(linux amd64) at http://fileserver.pingcap.net/download/builds/pingcap/test/tiflash/6b7b2567ab10ae9d4e0c4f18e5e7feb3caad5835/centos7/tiflash-linux-amd64.tar.gz |
What problem does this PR solve?
Issue Number: close #7564
Problem Summary:
Take over #7088
On KVStore's side:
fn_try_flush_data
.compactLogByRowKeyRange
when there is a need for flush.notifyCompactLog
to inform proxy.On Proxy's side:
notifyCompactLog
schedule_compact_log_task
, send aCheckCompact
request to ApplyRouter.check_pending_compact_log
which will persist the idealtruncated_state
from TiFlash.What is changed and how it works?
Check List
Tests
Side effects
Documentation
Release note