Horton

Re: Remove a redundant condition check

Michael Paquier <michael@paquier.xyz>

View thread View on postgr.es
From: Michael Paquier <michael@paquier.xyz>
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: Ádám Balogh <adam.balogh@ericsson.com>, PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2020-06-26T10:02:39Z
Lists: pgsql-hackers 
On Fri, Jun 26, 2020 at 02:39:22PM +0530, Amit Kapila wrote:
> It seems we forgot to remove the additional check for switchedTLI
> while adding a new check.  I think we can remove this duplicate check
> in the HEAD code.  I am not sure if it is worth to backpatch such a
> change.

Yes, there is no point to keep this check so let's clean up this
code.  I also see no need to do a backpatch here, this is purely
cosmetic.
--
Michael

Commits

  1. Remove duplicate check added by commit b2a5545bd6.

  2. Don't archive bogus recycled or preallocated files after timeline switch.