Quanlong Huang 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 13:

(1 comment)

> Patch Set 13: Verified-1
>
> Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/8026/

I take a look at file-list-begin-1.log in one of the build. There are no files 
under the functional_parquet.alltypes_transactional and 
functional_parquet.jointbl_transactional tables. Neither can I find statements 
loading these two tables in 
logs/data_loading/sql/functional/load-functional-query-exhaustive-impala-generated-parquet-none-none.sql.

Note that these tables under the functional db is ok (non-empty). However, I 
can't find the rebuild statement on the MV in 
logs/data_loading/sql/functional/load-functional-query-exhaustive-hive-generated-text-none-none.sql.

http://gerrit.cloudera.org:8080/#/c/17595/11/fe/src/main/java/org/apache/impala/analysis/StmtMetadataLoader.java
File fe/src/main/java/org/apache/impala/analysis/StmtMetadataLoader.java:

http://gerrit.cloudera.org:8080/#/c/17595/11/fe/src/main/java/org/apache/impala/analysis/StmtMetadataLoader.java@322
PS11, Line 322:         ((MaterializedViewHdfsTable) 
tbl).addSrcTables(mvSrcTableNames);
> MVs are not allowed to be defined on regular views since MVs expect the sou
I see. Thanks for the explanation!



--
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: 13
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: Fri, 08 Apr 2022 08:07:27 +0000
Gerrit-HasComments: Yes

Reply via email to