out of date comment in commit_ts.c - Mailing list pgsql-hackers

From Nathan Bossart
Subject out of date comment in commit_ts.c
Date
Msg-id 20220726173343.GA154110@nathanxps13
Whole thread Raw
Responses Re: out of date comment in commit_ts.c
List pgsql-hackers
Hi hackers,

I noticed that commit_ts.c has the following comment:

 * XLOG interactions: this module generates an XLOG record whenever a new
 * CommitTs page is initialized to zeroes.  Also, one XLOG record is
 * generated for setting of values when the caller requests it; this allows
 * us to support values coming from places other than transaction commit.
 * Other writes of CommitTS come from recording of transaction commit in
 * xact.c, which generates its own XLOG records for these events and will
 * re-perform the status update on redo; so we need make no additional XLOG
 * entry here.

IIUC the ability for callers to request WAL record generation is no longer
possible as of 08aa89b [0].  Should the second sentence be removed?

[0] https://git.postgresql.org/gitweb/?p=postgresql.git;a=commit;h=08aa89b

-- 
Nathan Bossart
Amazon Web Services: https://aws.amazon.com



pgsql-hackers by date:

Previous
From: Justin Pryzby
Date:
Subject: Re: Password reset link / 'less' does not exit in psql version 13.4
Next
From: Robert Haas
Date:
Subject: Re: predefined role(s) for VACUUM and ANALYZE