RelationPurgeLocalRelation():

            /*
             * RelationFlushRelation () below will flush relation information
             * from the cache. We must call smgrclose to flush relation
             * information from SMGR & FMGR, too. We assume that for temp
             * relations smgrunlink is already called by heap_destroyr
             * and we skip smgrclose for them.          - vadim 05/22/97
             */
            smgrclose(reln->rd_rel->relsmgr, reln);

 - it avoids memory leaks in SMGR & VFD.

RelationFlushRelation():
   there is no more call FileInvalidate(RelationGetSystemPort(relation));
   - invalid (FileInvalidate() expects File, not SMGR' fd)
   - unuseful anyway.

diff --git a/src/backend/utils/cache/relcache.c b/src/backend/utils/cache/relcache.c
index 31c8f35214456dc381ec5923717b819762128be2..61c6359e0885a9370fe057d9cb4522e0c01d1e3a 100644 (file)
--- a/src/backend/utils/cache/relcache.c
+++ b/src/backend/utils/cache/relcache.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *    $Header: /cvsroot/pgsql/src/backend/utils/cache/relcache.c,v 1.7 1997/05/20 11:41:38 vadim Exp $
+ *    $Header: /cvsroot/pgsql/src/backend/utils/cache/relcache.c,v 1.8 1997/05/22 17:24:20 vadim Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1217,9 +1217,7 @@ RelationFlushRelation(Relation *relationPtr,
    oldcxt = MemoryContextSwitchTo((MemoryContext)CacheCxt);
    
    RelationCacheDelete(relation);
-   
-   FileInvalidate(RelationGetSystemPort(relation));
-   
+
    p = relation->rd_att->attrs;
    for (i = 0; i < relation->rd_rel->relnatts; i++, p++)
        pfree (*p);
@@ -1425,6 +1423,15 @@ RelationPurgeLocalRelation(bool xactCommitted)
        smgrunlink(reln->rd_rel->relsmgr, reln);
        }
    }
+   else if ( !IsBootstrapProcessingMode () && !(reln->rd_istemp) )
+       /*
+        * RelationFlushRelation () below will flush relation information
+        * from the cache. We must call smgrclose to flush relation
+        * information from SMGR & FMGR, too. We assume that for temp
+        * relations smgrunlink is already called by heap_destroyr
+        * and we skip smgrclose for them.      - vadim 05/22/97
+        */
+       smgrclose(reln->rd_rel->relsmgr, reln);
    
    reln->rd_islocal = FALSE;