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
Enhancement CBO #2563
Enhancement CBO #2563
Conversation
[REVIEW NOTIFICATION] This pull request has been approved by:
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. |
/run-all-tests |
/run-all-tests test-flash=true |
/run-all-tests test-flash=true |
/run-all-tests |
tikv-client/src/main/java/com/pingcap/tikv/predicates/TiKVScanAnalyzer.java
Outdated
Show resolved
Hide resolved
tikv-client/src/main/java/com/pingcap/tikv/predicates/TiKVScanAnalyzer.java
Outdated
Show resolved
Hide resolved
tikv-client/src/main/java/com/pingcap/tikv/predicates/TiKVScanAnalyzer.java
Outdated
Show resolved
Hide resolved
…Analyzer.java Co-authored-by: Leon Yang <yangxin@pingcap.com>
…Analyzer.java Co-authored-by: Leon Yang <yangxin@pingcap.com>
tikv-client/src/main/java/com/pingcap/tikv/predicates/TiKVScanAnalyzer.java
Outdated
Show resolved
Hide resolved
…Analyzer.java Co-authored-by: Leon Yang <yangxin@pingcap.com>
tikv-client/src/main/java/com/pingcap/tikv/predicates/TiKVScanAnalyzer.java
Outdated
Show resolved
Hide resolved
tikv-client/src/main/java/com/pingcap/tikv/predicates/TiKVScanAnalyzer.java
Outdated
Show resolved
Hide resolved
tikv-client/src/main/java/com/pingcap/tikv/predicates/TiKVScanAnalyzer.java
Outdated
Show resolved
Hide resolved
tikv-client/src/main/java/com/pingcap/tikv/predicates/TiKVScanAnalyzer.java
Outdated
Show resolved
Hide resolved
…Analyzer.java Co-authored-by: Daemonxiao <35677990+Daemonxiao@users.noreply.github.com>
…Analyzer.java Co-authored-by: Daemonxiao <35677990+Daemonxiao@users.noreply.github.com>
…Analyzer.java Co-authored-by: Daemonxiao <35677990+Daemonxiao@users.noreply.github.com>
…Analyzer.java Co-authored-by: Daemonxiao <35677990+Daemonxiao@users.noreply.github.com>
/run-all-tests |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: b33b70e
|
cherry pick to release-release failed |
In response to a cherrypick label: new pull request created: #2566. |
Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
In response to a cherrypick label: new pull request created: #2567. |
Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
In response to a cherrypick label: new pull request created: #2568. |
In response to a cherrypick label: new pull request created: #2569. |
Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
What problem does this PR solve?
After this PR, TiSpark will choose the store according to the min cost between tikv table scan, tikv index scan and tifalsh scan.
Before this pr, TiSpark only compare tikv table scan and tiflash scan, and usually choose tiflash once the table has tiflash replica.