pengdou has posted comments on this change. ( http://gerrit.cloudera.org:8080/19109 )
Change subject: IMPALA-11390: Describe formatted statement on materialized view should show the view definition ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/19109/2/fe/src/main/java/org/apache/impala/util/HiveMetadataFormatUtils.java File fe/src/main/java/org/apache/impala/util/HiveMetadataFormatUtils.java: http://gerrit.cloudera.org:8080/#/c/19109/2/fe/src/main/java/org/apache/impala/util/HiveMetadataFormatUtils.java@544 PS2, Line 544: formatOutput("View Original Text:", tbl.getViewOriginalText(), tableInfo); > should it be `Materialized View Text`? This name is consistent with the hive materialized view, so "View Original Text:" might be more reasonable -- To view, visit http://gerrit.cloudera.org:8080/19109 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie62be1ca6d74370bad4b32bb046df93027e6f651 Gerrit-Change-Number: 19109 Gerrit-PatchSet: 2 Gerrit-Owner: pengdou <pengdou1...@126.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Jian Zhang <zjsar...@gmail.com> Gerrit-Reviewer: pengdou <pengdou1...@126.com> Gerrit-Comment-Date: Sun, 09 Oct 2022 11:36:29 +0000 Gerrit-HasComments: Yes
