From: | Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> |
---|---|
To: | Artur Zakirov <zaartur(at)gmail(dot)com> |
Cc: | Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, nagata(at)sraoss(dot)co(dot)jp |
Subject: | Re: Checking MINIMUM_VERSION_FOR_WAL_SUMMARIES |
Date: | 2024-02-02 12:11:40 |
Message-ID: | CAN55FZ3srLuGjAw1HgZjp7SrhF87E8Q+12G_cyAwifFZ4Um+tQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
On Fri, 2 Feb 2024 at 12:11, Artur Zakirov <zaartur(at)gmail(dot)com> wrote:
>
> On Fri, 2 Feb 2024 at 09:41, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | shveta malik | 2024-02-02 12:16:04 | Re: Synchronizing slots from primary to standby |
Previous Message | Yugo NAGATA | 2024-02-02 11:56:08 | Re: InstallXLogFileSegment() vs concurrent WAL flush |