Set updateType once for Materialized views refresh

trinodb · Oct 8, 2021 · 85b6419 · 85b6419
1 parent d39544e
commit 85b6419
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/core/trino-main/src/main/java/io/trino/sql/analyzer/StatementAnalyzer.java b/core/trino-main/src/main/java/io/trino/sql/analyzer/StatementAnalyzer.java
@@ -526,10 +526,10 @@ protected Scope visitRefreshMaterializedView(RefreshMaterializedView refreshMate
             }
 
             accessControl.checkCanRefreshMaterializedView(session.toSecurityContext(), name);
+            analysis.setUpdateType("REFRESH MATERIALIZED VIEW");
 
             if (metadata.delegateMaterializedViewRefreshToConnector(session, name)) {
                 analysis.setDelegatedRefreshMaterializedView(name);
-                analysis.setUpdateType("REFRESH MATERIALIZED VIEW");
                 analysis.setUpdateTarget(
                         name,
                         Optional.empty(),
@@ -590,7 +590,6 @@ protected Scope visitRefreshMaterializedView(RefreshMaterializedView refreshMate
                             .map(Type::toString),
                     Column::new);
 
-            analysis.setUpdateType("REFRESH MATERIALIZED VIEW");
             analysis.setUpdateTarget(
                     targetTable,
                     Optional.empty(),