Skip to content

Conversation

@yangzhg
Copy link
Member

@yangzhg yangzhg commented Jun 9, 2020

implemnets #3803
Support disable some unmeaningful order by clause.
The default limit of 65535 will not be disabled because of it is added at plannode, after we support spill to disk we can move this limit to analyze

@yangzhg yangzhg force-pushed the improve_performance branch from 3d0bdbb to 85bc7f7 Compare June 10, 2020 11:29
@chaoyli
Copy link
Contributor

chaoyli commented Jun 10, 2020

why it has much of uncorrelated code.

@yangzhg yangzhg force-pushed the improve_performance branch from 85bc7f7 to 5377c24 Compare June 16, 2020 04:09
@yangzhg
Copy link
Member Author

yangzhg commented Jun 16, 2020

why it has much of uncorrelated code.

done

@morningman morningman added area/planner Issues or PRs related to the query planner kind/improvement labels Jun 18, 2020
@yangzhg yangzhg force-pushed the improve_performance branch from 9acf251 to d0c6e9d Compare June 22, 2020 05:41
@yangzhg yangzhg changed the title Add a session variable allow remove order by for subquery in set opertion clause 人、emove order by for subquery in set opertion clause Jun 24, 2020
@yangzhg yangzhg changed the title 人、emove order by for subquery in set opertion clause Remove order by for subquery in set opertion clause Jun 24, 2020
@yangzhg yangzhg force-pushed the improve_performance branch 3 times, most recently from e7d8c27 to ddf26f0 Compare June 30, 2020 01:56
@yangzhg yangzhg force-pushed the improve_performance branch from ddf26f0 to 25cbe16 Compare July 1, 2020 07:18
Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@morningman morningman added the approved Indicates a PR has been approved by one committer. label Jul 1, 2020
@morningman morningman merged commit 707d03c into apache:master Jul 2, 2020
@yangzhg yangzhg deleted the improve_performance branch July 3, 2020 06:34
morningman pushed a commit to morningman/doris that referenced this pull request Jul 20, 2020
implemnets apache#3803 
Support disable some unmeaningful order by clause.
The default limit of 65535 will not be disabled because of it is added at plannode,
after we support spill to disk we can move this limit to analyze.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. area/planner Issues or PRs related to the query planner kind/improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants