From: | Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pg_wal_summary_contents() and pg_walsummary may return different results on the same WAL summary file |
Date: | 2024-07-04 10:16:16 |
Message-ID: | 71c59c8d-35f1-46a9-adba-c614ec001898@oss.nttdata.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2024/07/03 22:42, Robert Haas wrote:
> On Wed, Jul 3, 2024 at 5:34 AM Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> wrote:
>> pg_wal_summary_contents() seems to miss the summary information with "limit" that pg_walsummary reports. This appears to be a bug. The attached patch fixes this.
>
> Oops. It looks like pg_wal_summary_contents() forgets to emit the
> limit block when that's the only data for a particular relation fork.
> And maybe you can make it emit the limit block multiple times if the
> list of block numbers is long enough.
>
> Thanks for the patch. I think you can commit and back-patch this, but
> I don't think the commit message is quite right, because it's not like
> this code just NEVER executes where it is located currently. Or am I
> missing something?
Yes, so I updated the commit message. I borrowed your description and used it in the message. Attached is the revised version of the patch.
If there are no objections, I will commit and backpatch it.
Regards,
--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION
Attachment | Content-Type | Size |
---|---|---|
v2-0001-Fix-bugs-in-pg_wal_summary_contents.patch | text/plain | 2.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Mohab Yaser | 2024-07-04 10:16:17 | Problem while installing PostgreSQL using make |
Previous Message | Aleksander Alekseev | 2024-07-04 10:10:35 | Re: Grammar guidelines in Postgres |