Re: Remove a redundant condition check

Amit Kapila <amit.kapila16@gmail.com>

From: Amit Kapila <amit.kapila16@gmail.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Ádám Balogh <adam.balogh@ericsson.com>, PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2020-06-27T05:22:49Z
Lists: pgsql-hackers
On Fri, Jun 26, 2020 at 3:32 PM Michael Paquier <michael@paquier.xyz> wrote:
>
> 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.
>

Thanks for the confirmation, pushed!

-- 
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com



Commits

  1. Remove duplicate check added by commit b2a5545bd6.

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