Date: Mon, 1 Aug 2022 22:02:19 -0700
From: Nathan Bossart <nathandbossart@gmail.com>
To: Noah Misch <noah@leadboat.com>
Cc: David Steele <david@pgmasters.net>,
	Kyotaro Horiguchi <horikyota.ntt@gmail.com>, robertmhaas@gmail.com,
	masao.fujii@oss.nttdata.com, pgsql-hackers@postgresql.org
Subject: Re: remove more archiving overhead
Message-ID: <20220802050219.GA475090@nathanxps13>
References: <20220707182251.GA2255242@nathanxps13>
 <ba7e11b0-054d-1d26-bafd-7a2bc97f11f7@pgmasters.net>
 <20220707220716.GA2258527@nathanxps13>
 <20220708.105652.1551158050191356887.horikyota.ntt@gmail.com>
 <44dbf5e4-8eec-da56-daba-755128b3723e@pgmasters.net>
 <20220708165450.GA2356733@nathanxps13>
 <20220731065156.GA3694276@rfd.leadboat.com>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="opJtzjQTFsWo+cga"
Content-Disposition: inline
In-Reply-To: <20220731065156.GA3694276@rfd.leadboat.com>
Archived-At: 
 <https://www.postgresql.org/message-id/20220802050219.GA475090%40nathanxps13>
Precedence: bulk


--opJtzjQTFsWo+cga
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Sat, Jul 30, 2022 at 11:51:56PM -0700, Noah Misch wrote:
> Inviting the administrator to resolve things is more dangerous than just
> returning true.  I recommend making this text more opinionated and simpler:
> libraries must return true.  Alternately, if some library has found a good
> reason to return false, this paragraph could give the reason.  I don't know of
> such a reason, though.

Your suggestion seems reasonable to me.  I've attached a small patch.

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

--opJtzjQTFsWo+cga
Content-Type: text/x-diff; charset=us-ascii
Content-Disposition: attachment; filename="adjust_archive_docs.patch"

diff --git a/doc/src/sgml/backup.sgml b/doc/src/sgml/backup.sgml
index e432bb015a..cba7292f98 100644
--- a/doc/src/sgml/backup.sgml
+++ b/doc/src/sgml/backup.sgml
@@ -691,11 +691,9 @@ test ! -f /mnt/server/archivedir/00000001000000A900000065 &amp;&amp; cp pg_wal/0
     system crashes before the server makes a durable record of archival success,
     the server will attempt to archive the file again after restarting (provided
     archiving is still enabled).  When an archive library encounters a
-    pre-existing file, it may return <literal>true</literal> if the WAL file has
+    pre-existing file, it should return <literal>true</literal> if the WAL file has
     identical contents to the pre-existing archive and the pre-existing archive
-    is fully persisted to storage.  Alternatively, the archive library may
-    return <literal>false</literal> anytime a pre-existing file is encountered,
-    but this will require manual action by an administrator to resolve.  If a
+    is fully persisted to storage.  If a
     pre-existing file contains different contents than the WAL file being
     archived, the archive library <emphasis>must</emphasis> return
     <literal>false</literal>.

--opJtzjQTFsWo+cga--