Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rUEa4-009hAI-Bf for pgsql-hackers@arkaria.postgresql.org; Sun, 28 Jan 2024 23:28:52 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.94.2) (envelope-from ) id 1rUEa3-00AJ11-1i for pgsql-hackers@arkaria.postgresql.org; Sun, 28 Jan 2024 23:28:51 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rUEa2-00AJ0t-OX for pgsql-hackers@lists.postgresql.org; Sun, 28 Jan 2024 23:28:50 +0000 Received: from mail.thelabyrinth.net ([45.56.70.56]) by magus.postgresql.org with esmtps (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rUEZz-004EOP-Tc for pgsql-hackers@lists.postgresql.org; Sun, 28 Jan 2024 23:28:50 +0000 Received: from [10.5.0.2] (unknown [5.182.32.137]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: dsteele) by mail.thelabyrinth.net (Postfix) with ESMTPSA id 2DD7754667; Sun, 28 Jan 2024 23:28:43 +0000 (UTC) Message-ID: Date: Sun, 28 Jan 2024 19:28:41 -0400 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: Add recovery to pg_control and remove backup_label To: Michael Paquier , vignesh C Cc: Robert Haas , Postgres hackers , Andres Freund , Stephen Frost References: <77ad92e7-3e37-40e2-8e3b-df3cf0108aa8@pgmasters.net> Content-Language: en-US From: David Steele In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk On 1/28/24 19:11, Michael Paquier wrote: > On Fri, Jan 26, 2024 at 06:27:30PM +0530, vignesh C wrote: >> Please post an updated version for the same. >> >> [1] - http://cfbot.cputube.org/patch_46_3511.log > > With the recent introduction of incremental backups that depend on > backup_label and the rather negative feedback received, I think that > it would be better to return this entry as RwF for now. What do you > think? I've been thinking it makes little sense to update the patch. It would be a lot of work with all the new changes for incremental backup and since Andres and Robert appear to be very against the idea, I doubt it would be worth the effort. I have withdrawn the patch. Regards, -David