Fix cache flush hazard in ExecRefreshMatView.

Andres Freund

diff --git a/src/backend/commands/matview.c b/src/backend/commands/matview.c
index 46149eed4f7c63517f6a4bb51825debb4cad87f6..09ea344256a49c3613856f45ad4e447aa0a1aa7f 100644 (file)
--- a/src/backend/commands/matview.c
+++ b/src/backend/commands/matview.c
@@ -141,6 +141,7 @@ ExecRefreshMatView(RefreshMatViewStmt *stmt, const char *queryString,
    List       *actions;
    Query      *dataQuery;
    Oid         tableSpace;
+   Oid         owner;
    Oid         OIDNewHeap;
    DestReceiver *dest;
    bool        concurrent;
@@ -238,6 +239,8 @@ ExecRefreshMatView(RefreshMatViewStmt *stmt, const char *queryString,
    else
        tableSpace = matviewRel->rd_rel->reltablespace;
 
+   owner = matviewRel->rd_rel->relowner;
+
    heap_close(matviewRel, NoLock);
 
    /* Create the transient table that will receive the regenerated data. */
@@ -247,8 +250,7 @@ ExecRefreshMatView(RefreshMatViewStmt *stmt, const char *queryString,
 
    /* Generate the data, if wanted. */
    if (!stmt->skipData)
-       refresh_matview_datafill(dest, dataQuery, queryString,
-                                matviewRel->rd_rel->relowner);
+       refresh_matview_datafill(dest, dataQuery, queryString, owner);
 
    /* Make the matview match the newly generated data. */
    if (concurrent)