Skip to content

[refactor](Nereids) mv top-n two phase read rule from post processor to rewriter - #22894

Merged
xiaokang merged 1 commit into
apache:branch-2.0from
morrySnow:branch-2.0-topn
Aug 11, 2023
Merged

[refactor](Nereids) mv top-n two phase read rule from post processor to rewriter#22894
xiaokang merged 1 commit into
apache:branch-2.0from
morrySnow:branch-2.0-topn

Conversation

@morrySnow

Copy link
Copy Markdown
Contributor

Proposed changes

Issue Number: close #xxx

cherry-pick from master #22487
commit id: 3961b8d

use three new plan node to represent defer materialize of TopN.
Example:

```
-- SQL
select * from t1 order by c1 limit 10;
-- PLAN
+------------------------------------------+
| Explain String |
+------------------------------------------+
| PhysicalDeferMaterializeResultSink |
| --PhysicalDeferMaterializeTopN |
| ----PhysicalDistribute |
| ------PhysicalDeferMaterializeTopN |
| --------PhysicalDeferMaterializeOlapScan |
+------------------------------------------+
```

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

…to rewriter (apache#22487)
use three new plan node to represent defer materialize of TopN.
Example:
```
-- SQL
select * from t1 order by c1 limit 10;
-- PLAN
+------------------------------------------+
| Explain String |
+------------------------------------------+
| PhysicalDeferMaterializeResultSink |
| --PhysicalDeferMaterializeTopN |
| ----PhysicalDistribute |
| ------PhysicalDeferMaterializeTopN |
| --------PhysicalDeferMaterializeOlapScan |
+------------------------------------------+
```
@github-actionsgithub-actionsBot added area/nereids area/planner Issues or PRs related to the query planner kind/docs Categorizes issue or PR as related to documentation. labels Aug 11, 2023
@xiaokang
xiaokang merged commit 4ef0f53 into apache:branch-2.0Aug 11, 2023
@morrySnow
morrySnow deleted the branch-2.0-topn branch September 4, 2023 02:44
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/nereidsarea/plannerIssues or PRs related to the query plannerdev/2.0.1-mergedkind/docsCategorizes issue or PR as related to documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@morrySnow@xiaokang