Fix cosmetic blemishes involving rd_createSubid.

Remove an obsolete comment from AtEOXact_cleanup().  Restore formatting
of a comment in struct RelationData, mangled by the pgindent run in
commit 9af4159fce6654aa0e081b00d02bca40b978745c.  Back-patch to 9.5 (all
supported versions), because another fix stacks on this.

diff --git a/src/backend/utils/cache/relcache.c b/src/backend/utils/cache/relcache.c
index 9d017037919c491e7c89513b98532da3e4884777..462b1a9636a69a86a2f60621e39653aebc66e29b 100644 (file)
--- a/src/backend/utils/cache/relcache.c
+++ b/src/backend/utils/cache/relcache.c
@@ -2926,10 +2926,7 @@ AtEOXact_cleanup(Relation relation, bool isCommit)
     *
     * During commit, reset the flag to zero, since we are now out of the
     * creating transaction.  During abort, simply delete the relcache entry
-    * --- it isn't interesting any longer.  (NOTE: if we have forgotten the
-    * new-ness of a new relation due to a forced cache flush, the entry will
-    * get deleted anyway by shared-cache-inval processing of the aborted
-    * pg_class insertion.)
+    * --- it isn't interesting any longer.
     */
    if (relation->rd_createSubid != InvalidSubTransactionId)
    {

diff --git a/src/include/utils/rel.h b/src/include/utils/rel.h
index 756cd56d2e257dc6fc8fd95dbdf2782dce90d013..11a635964fd0861357059c8519e42dd90dda497a 100644 (file)
--- a/src/include/utils/rel.h
+++ b/src/include/utils/rel.h
@@ -64,7 +64,7 @@ typedef struct RelationData
                                 * valid, 2 = temporarily forced */
    bool        rd_statvalid;   /* is rd_statlist valid? */
 
-   /*
+   /*----------
     * rd_createSubid is the ID of the highest subtransaction the rel has
     * survived into; or zero if the rel was not created in the current top
     * transaction.  This can be now be relied on, whereas previously it could
@@ -74,8 +74,13 @@ typedef struct RelationData
     * have forgotten changing it). rd_newRelfilenodeSubid can be forgotten
     * when a relation has multiple new relfilenodes within a single
     * transaction, with one of them occurring in a subsequently aborted
-    * subtransaction, e.g. BEGIN; TRUNCATE t; SAVEPOINT save; TRUNCATE t;
-    * ROLLBACK TO save; -- rd_newRelfilenode is now forgotten
+    * subtransaction, e.g.
+    *      BEGIN;
+    *      TRUNCATE t;
+    *      SAVEPOINT save;
+    *      TRUNCATE t;
+    *      ROLLBACK TO save;
+    *      -- rd_newRelfilenodeSubid is now forgotten
     */
    SubTransactionId rd_createSubid;    /* rel was created in current xact */
    SubTransactionId rd_newRelfilenodeSubid;    /* new relfilenode assigned in