Skip to content

Conversation

@HangyuanLiu
Copy link
Contributor

@HangyuanLiu HangyuanLiu commented Jun 15, 2020

  • Building the materialized view function for schema_change here based on defineExpr.
  • This is a trick because the current storage layer does not support expression evaluation.
  • count distinct materialized view will set mv_expr with to_bitmap or hll_hash.
  • count materialized view will set mv_expr with count.
  • Support to regenerate historical data when a new materialized view is created in BE。
    1. Support to_bitmap function
    2. Support hll_hash function
    3. Support count(field) function
      For [Proposal] Materialized View 2.0 #3344

@EmmyMiao87
Copy link
Contributor

Please provide detailed commit message

@HangyuanLiu
Copy link
Contributor Author

Please provide detailed commit message

Done

@EmmyMiao87 EmmyMiao87 added area/materialized-view Issues or PRs related to materialized view kind/feature Categorizes issue or PR as related to a new feature. labels Jul 2, 2020
add create materialized view in rollup v2
 into mv_func

� Conflicts:
�	be/src/olap/schema_change.cpp
# Conflicts:
#	be/src/olap/schema_change.cpp
#	be/src/olap/schema_change.h
Copy link
Contributor

@EmmyMiao87 EmmyMiao87 left a comment

Choose a reason for hiding this comment

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

LGTM

@EmmyMiao87 EmmyMiao87 added the approved Indicates a PR has been approved by one committer. label Jul 14, 2020
@EmmyMiao87 EmmyMiao87 merged commit 5032b7f into apache:master Jul 16, 2020
@EmmyMiao87 EmmyMiao87 mentioned this pull request Sep 1, 2020
englefly pushed a commit to englefly/incubator-doris that referenced this pull request May 6, 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/feature Categorizes issue or PR as related to a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants