Skip to content

Conversation

@EmmyMiao87
Copy link
Contributor

@EmmyMiao87 EmmyMiao87 commented Jul 9, 2020

Fix bug of count(*) in MV selector

The output columns of query should be collected by all of tupleIds in BaseTableRef rather than the top tupleIds of query.
The top tupleIds of count(*) is Agg tuple which does not expand the star.

Fixed #4065

Change-Id: Icd21a10f505e01fefba30c8dfce8a0fa162cffa6

@EmmyMiao87 EmmyMiao87 added kind/fix Categorizes issue or PR as related to a bug. area/materialized-view Issues or PRs related to materialized view labels Jul 9, 2020
The output columns of query should be collected by all of tupleIds in BaseTableRef rather than the top tupleIds of query.
The top tupleIds of count(*) is Agg tuple which does not expand the star.

Fixed#4065
Change-Id: Icd21a10f505e01fefba30c8dfce8a0fa162cffa6
Copy link
Member

@yangzhg yangzhg 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 10, 2020
@morningman morningman merged commit e435e6f into apache:master Jul 13, 2020
morningman pushed a commit to morningman/doris that referenced this pull request Jul 16, 2020
The output columns of query should be collected by all of tupleIds
in BaseTableRef rather than the top tupleIds of query.
The top tupleIds of count(*) is Agg tuple which does not expand the star.

Fixed apache#4065
deardeng added a commit to deardeng/incubator-doris that referenced this pull request Jul 8, 2025
Hastyshell pushed a commit to Hastyshell/doris that referenced this pull request Jul 21, 2025
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/materialized-view Issues or PRs related to materialized view kind/fix Categorizes issue or PR as related to a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Materialized selector chooses the error index when Count(*)

3 participants