Re: Checking MINIMUM_VERSION_FOR_WAL_SUMMARIES - Mailing list pgsql-hackers

From Artur Zakirov
Subject Re: Checking MINIMUM_VERSION_FOR_WAL_SUMMARIES
Date
Msg-id CAKNkYnzO4SAr-8Ja9YUgCteHPYWsRdJk6w4aXp7ziao7Ms4hhw@mail.gmail.com
Whole thread Raw
In response to Re: Checking MINIMUM_VERSION_FOR_WAL_SUMMARIES  (Nazir Bilal Yavuz <byavuz81@gmail.com>)
Responses Re: Checking MINIMUM_VERSION_FOR_WAL_SUMMARIES
List pgsql-hackers
On Fri, 2 Feb 2024 at 09:41, Nazir Bilal Yavuz <byavuz81@gmail.com> wrote:
> You seem right, nice catch. Also, this change makes the check in
>
>             snprintf(summarydir, sizeof(summarydir), "%s/%s/summaries",
>                      basedir,
>                      PQserverVersion(conn) < MINIMUM_VERSION_FOR_PG_WAL ?
>                      "pg_xlog" : "pg_wal");
>
> redundant. PQserverVersion(conn) will always be higher than
> MINIMUM_VERSION_FOR_PG_WAL.

Thank you both for the comments. Indeed, that part now looks redundant.
I've attached a patch to remove checking MINIMUM_VERSION_FOR_PG_WAL.

-- 
Artur

Attachment

pgsql-hackers by date:

Previous
From: Alexander Lakhin
Date:
Subject: Re: ResourceOwner refactoring
Next
From: Richard Guo
Date:
Subject: Re: Check lateral references within PHVs for memoize cache keys