Re: Checking MINIMUM_VERSION_FOR_WAL_SUMMARIES - Mailing list pgsql-hackers

From Nazir Bilal Yavuz
Subject Re: Checking MINIMUM_VERSION_FOR_WAL_SUMMARIES
Date
Msg-id CAN55FZ3srLuGjAw1HgZjp7SrhF87E8Q+12G_cyAwifFZ4Um+tQ@mail.gmail.com
Whole thread Raw
In response to Re: Checking MINIMUM_VERSION_FOR_WAL_SUMMARIES  (Artur Zakirov <zaartur@gmail.com>)
Responses Re: Checking MINIMUM_VERSION_FOR_WAL_SUMMARIES
List pgsql-hackers
Hi,

On Fri, 2 Feb 2024 at 12:11, Artur Zakirov <zaartur@gmail.com> wrote:
>
> 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.

Thanks for the update. The patch looks good to me.

-- 
Regards,
Nazir Bilal Yavuz
Microsoft



pgsql-hackers by date:

Previous
From: Yugo NAGATA
Date:
Subject: Re: InstallXLogFileSegment() vs concurrent WAL flush
Next
From: shveta malik
Date:
Subject: Re: Synchronizing slots from primary to standby