pgsql: Update high level vacuumlazy.c comments.

From: Peter Geoghegan <pg(at)bowt(dot)ie>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Update high level vacuumlazy.c comments.
Date: 2021-11-27 22:31:24
Message-ID: E1mr6E8-0000x1-SN@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Update high level vacuumlazy.c comments.

Update vacuumlazy.c file header comments (as well as comments above the
lazy_scan_heap function) that were largely written before the
introduction of the HOT optimization, when lazy_scan_heap did far less,
and didn't actually prune during its initial heap pass.

Since lazy_scan_heap now outsources far more work to lower level
functions, it makes sense to introduce the function by talking about the
high level invariant that dictates the order in which each phase takes
place. Also deemphasize the case where we run out of memory for TIDs,
since delaying that discussion makes it easier to talk about issues of
central importance.

Finally, remove discussion of parallel VACUUM from header comments.
These don't add much, and are in the wrong place.

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/12b5ade9023f3ecaddcbc423a22dc284c91c79f6

Modified Files
--------------
src/backend/access/heap/vacuumlazy.c | 138 ++++++++++++++++++-----------------
1 file changed, 70 insertions(+), 68 deletions(-)

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Geoghegan 2021-11-28 00:06:22 pgsql: vacuumlazy.c: prefer the term "cleanup lock".
Previous Message Andres Freund 2021-11-27 01:55:17 Re: pgsql: xlog.c: Remove global variables ReadRecPtr and EndRecPtr.