Qifan Chen has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17595 )

Change subject: IMPALA-10723: Treat materialized view as a table instead of a 
view
......................................................................


Patch Set 12:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/17595/7/testdata/workloads/functional-query/queries/QueryTest/compute-stats.test
File testdata/workloads/functional-query/queries/QueryTest/compute-stats.test:

http://gerrit.cloudera.org:8080/#/c/17595/7/testdata/workloads/functional-query/queries/QueryTest/compute-stats.test@1841
PS7, Line 1841: G
> Adding the MV syntax is not in scope for this patch because the DDLs for MV
Thanks for the explanation.

I still feel it is useful to distinguish tables from MVs at SQL level. What 
HIVE does in DDLs for MVs are right. If we can carry it to the IMPALA commands 
on MVs (such as show stats in this patch), we maintain that property.



--
To view, visit http://gerrit.cloudera.org:8080/17595
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If3108996124c6544a97fb0c34b6aff5e324a6cff
Gerrit-Change-Number: 17595
Gerrit-PatchSet: 12
Gerrit-Owner: Aman Sinha <amsi...@cloudera.com>
Gerrit-Reviewer: Aman Sinha <amsi...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Qifan Chen <qc...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Comment-Date: Thu, 24 Mar 2022 13:29:07 +0000
Gerrit-HasComments: Yes

Reply via email to