One more migration fix (#298)

Don't recreate reporting views before MV migration
sundeck-io · Sep 13, 2023 · 8087dad · 8087dad
1 parent c6bc328
commit 8087dad
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/bootstrap/019_query_history_updates.sql b/bootstrap/019_query_history_updates.sql
@@ -7,8 +7,8 @@ language sql
 as
 begin
     SYSTEM$LOG_TRACE('Migrating query history data.');
-    -- Update the internal_reporting views to reflect any updates from snowflake's views.
-    call internal.migrate_view();
+    -- Update _only_ the internal_reporting views to reflect any updates from snowflake's views.
+    call INTERNAL.create_view_QUERY_HISTORY_COMPLETE_AND_DAILY();
     let migrate1 string := '';
     call internal.migrate_if_necessary('INTERNAL_REPORTING', 'QUERY_HISTORY_COMPLETE_AND_DAILY', 'INTERNAL_REPORTING_MV', 'QUERY_HISTORY_COMPLETE_AND_DAILY') into :migrate1;
     let migrate2 string := '';

diff --git a/bootstrap/090_post_setup.sql b/bootstrap/090_post_setup.sql
@@ -66,7 +66,7 @@ CREATE OR REPLACE TASK TASKS.SFUSER_MAINTENANCE
     AS
     CALL INTERNAL.refresh_users();
 
--- enable the query_hash column in the query_history view
+-- create the query_hash functions in TOOLS
 call INTERNAL.ENABLE_QUERY_HASH();
 
 -- Populate the list of predefined labels