vacuumlazy.c: prefer the term "cleanup lock".
authorPeter Geoghegan <pg@bowt.ie>
Sun, 28 Nov 2021 00:05:01 +0000 (16:05 -0800)
committerPeter Geoghegan <pg@bowt.ie>
Sun, 28 Nov 2021 00:05:01 +0000 (16:05 -0800)
The term "super-exclusive lock" is an acceptable synonym of "cleanup
lock".  Even still, switching from one term to the other in the same
file is confusing.  Standardize on "cleanup lock" within vacuumlazy.c.

Per a complaint from Andres Freund.

src/backend/access/heap/vacuumlazy.c

index de7ca8de8588bb40f6d88e8d8056847f1ae37e3a..ddd0bb98756d0a4e71f3be69ebb4d6c1904f5913 100644 (file)
@@ -1274,8 +1274,8 @@ lazy_scan_heap(LVRelState *vacrel, VacuumParams *params, bool aggressive)
         * By here we definitely have enough dead_tuples space for whatever
         * LP_DEAD tids are on this page, we have the visibility map page set
         * up in case we need to set this page's all_visible/all_frozen bit,
-        * and we have a super-exclusive lock.  Any tuples on this page are
-        * now sure to be "counted" by this VACUUM.
+        * and we have a cleanup lock.  Any tuples on this page are now sure
+        * to be "counted" by this VACUUM.
         *
         * One last piece of preamble needs to take place before we can prune:
         * we need to consider new and empty pages.
@@ -1524,8 +1524,8 @@ lazy_scan_heap(LVRelState *vacrel, VacuumParams *params, bool aggressive)
        }
 
        /*
-        * Final steps for block: drop super-exclusive lock, record free space
-        * in the FSM
+        * Final steps for block: drop cleanup lock, record free space in the
+        * FSM
         */
        if (prunestate.has_lpdead_items && vacrel->do_index_vacuuming)
        {
@@ -2377,8 +2377,8 @@ lazy_vacuum_heap_rel(LVRelState *vacrel)
  * lazy_vacuum_heap_page() -- free page's LP_DEAD items listed in the
  *                       vacrel->dead_tuples array.
  *
- * Caller must have an exclusive buffer lock on the buffer (though a
- * super-exclusive lock is also acceptable).
+ * Caller must have an exclusive buffer lock on the buffer (though a full
+ * cleanup lock is also acceptable).
  *
  * tupindex is the index in vacrel->dead_tuples of the first dead tuple for
  * this page.  We assume the rest follow sequentially.  The return value is