-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[feat](nereids) support explain delete from clause #36782
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
Conversation
|
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
morrySnow
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- delete from dup/agg table still cannot explain, and will throw wrong error message, we should handle it by throw right error message.
- add explain update case to ensure all dml is ok
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/DeleteFromCommand.java
Show resolved
Hide resolved
fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java
Show resolved
Hide resolved
6dac736 to
2f81b9b
Compare
|
run buildall |
2f81b9b to
7d8699e
Compare
|
run buildall |
TPC-H: Total hot run time: 39338 ms |
TPC-DS: Total hot run time: 172807 ms |
ClickBench: Total hot run time: 30.37 s |
7d8699e to
4333231
Compare
|
run buildall |
4333231 to
c61547b
Compare
|
run buildall |
TPC-H: Total hot run time: 39805 ms |
TPC-DS: Total hot run time: 169872 ms |
ClickBench: Total hot run time: 30.12 s |
|
run p0 |
1 similar comment
|
run p0 |
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
## Proposed changes support explain like: explain delete from T where A=1 Issue Number: close #xxx <!--Describe your changes.--> (cherry picked from commit dc369cd)
## Proposed changes support explain like: explain delete from T where A=1 Issue Number: close #xxx <!--Describe your changes.-->
… in Nereids planner (#38751) ## Proposed changes 1. Fix nereids planner don't use partial update for delete statement on merge-on-write table introduced in #36782 2. add `IS_PARTIAL_UPDATE: true` in `OlapTableSink.getExplainString()` 3. don't convert the delete stmt to partial update if the table has cluster key.
… in Nereids planner (#38751) ## Proposed changes 1. Fix nereids planner don't use partial update for delete statement on merge-on-write table introduced in #36782 2. add `IS_PARTIAL_UPDATE: true` in `OlapTableSink.getExplainString()` 3. don't convert the delete stmt to partial update if the table has cluster key.
… in Nereids planner (apache#38751) ## Proposed changes 1. Fix nereids planner don't use partial update for delete statement on merge-on-write table introduced in apache#36782 2. add `IS_PARTIAL_UPDATE: true` in `OlapTableSink.getExplainString()` 3. don't convert the delete stmt to partial update if the table has cluster key.
Proposed changes
support explain like:
explain delete from T where A=1
Issue Number: close #xxx