> On Nov 29, 2023, at 01:18, Nathan Bossart wrote:
>
> External Email
>
> Here is a new version of the patch with feedback addressed.
>
> --
> Nathan Bossart
> Amazon Web Services: https://aws.amazon.com
Hi Nathan,
The patch looks good to me. With the context explained in the thread, the
Rebase the patch against the latest HEAD.
Regards,
Yong
slru_page_header_v4.patch
Description: slru_page_header_v4.patch
> On Nov 28, 2023, at 05:00, Nathan Bossart wrote:
>
> External Email
>
> Here's a v2 of the patch set in which I've attempted to address all
> feedback. I've also added a pg_write_membarrier_u* pair of functions that
> provide an easy way to write to an atomic variable with full barrier
> se
>> - New comments have been added to pg_upgrade to mention the SLRU
>> page header change as the reason for upgrading clog files.
>
> That seems reasonable, but were any alternatives discussed? Do we have
> consensus that this is the right thing to do?
In general, there are two approaches. Eith
Rebase the patch against the latest HEAD.
Regards,
Yong
slru_page_header_v5.patch
Description: slru_page_header_v5.patch
> On Mar 7, 2024, at 03:09, Stephen Frost wrote:
>
> External Email
>
> From: Stephen Frost
> Subject: Re: Proposal to add page headers to SLRU pages
> Date: March 7, 2024 at 03:09:59 GMT+8
> To: Alvaro Herrera
> Cc: "Li, Yong" , Aleksander Alekseev
>
> On Mar 9, 2024, at 05:22, Jeff Davis wrote:
>
> External Email
>
> On Wed, 2024-03-06 at 12:01 +, Li, Yong wrote:
>> Rebase the patch against the latest HEAD.
>
> The upgrade logic could use more comments explaining what's going on
> and why.
Hi all,
PostgreSQL currently maintains several data structures in the SLRU cache. The
current SLRU pages do not have any header, so it is impossible to checksum a
page and verify its integrity. It is very difficult to debug issues caused by
corrupted SLRU pages. Also, without a page header,
ready for review. All check-world tests
including tap tests passed with this patch.
Regards,
Yong
From: Robert Haas
Date: Friday, December 8, 2023 at 03:51
To: Debnath, Shawn
Cc: Andrey Borodin , PostgreSQL Hackers
, Aleksander Alekseev
, Li, Yong , Shyrabokau, Anton
, Bagga, Rishu
Subject
> This work is being done in file.c – it seems to me the proper way to
> proceed would be to continue writing on-disk upgrade logic here.
> Besides that this looks good to me, would like to hear what others have to
> say.
Thank you, Rishu for taking time to review the code. I've updated the pat
> On Jan 2, 2024, at 19:35, Aleksander Alekseev
> wrote:
>
> Thanks for the updated patch.
>
> cfbot seems to have some complaints regarding compiler warnings and
> also building the patch on Windows:
>
> http://cfbot.cputube.org/
Thanks for the information. Here is the updated patch.
Regard
> On Jun 10, 2024, at 16:01, Michael Paquier wrote:
>
> External Email
>
> From: Michael Paquier
> Subject: Re: Proposal to add page headers to SLRU pages
> Date: June 10, 2024 at 16:01:50 GMT+8
> To: Bertrand Drouvot
> Cc: "Li, Yong" , Jeff Davis
Hi PostgreSQL hackers,
For most access methods in PostgreSQL, the implementation of the access method
itself and the implementation of its WAL replay logic are organized in separate
source files. However, the HEAP access method is an exception. Both the
access method and the WAL replay logic
> On Jun 17, 2024, at 23:01, Melanie Plageman wrote:
>
> External Email
>
> On Mon, Jun 17, 2024 at 2:20 AM Li, Yong wrote:
>>
>> Hi PostgreSQL hackers,
>>
>> For most access methods in PostgreSQL, the implementation of the access
>> method it
> On Jun 18, 2024, at 20:42, Melanie Plageman wrote:
>
> External Email
>
> On Mon, Jun 17, 2024 at 9:12 PM Li, Yong wrote:
>>
>> As a newcomer, when I was walking through the code looking for WAL replay
>> related code, it was relatively easy for me to fi
Hi Kou,
I tried to follow the thread but had to skip quite some discussions in the
middle part of the thread. From what I read, it appears to me that there were a
lot of back-and-forth discussions on the specific implementation details (i.e.
do not touch existing format implementation), perform
> On Jul 23, 2024, at 09:54, Sutou Kouhei wrote:
>
>
> Here are my comments for your patch:
>
> 1. Could you create your patch by "git format-patch -vN master"
> or something? If you create your patch by "git format-patch",
> we can apply your patch by "git am XXX.patch".
>
Thanks for your
> On Jul 25, 2024, at 12:51, Sutou Kouhei wrote:
>
> Hi,
>
> THREAD SUMMARY:
Very nice summary.
>
> Implementation:
>
> The v18 patch set is the latest patch set. [6]
> It includes the following patches:
>
> 0001: This adds a basic feature (Copy{From,To}Routine)
> (This isn't enough
> I think that this proposal is reasonable but we need to get
> attention from a committer to move forward this proposal.
>
>
> Thanks,
> —
> kou
Thank you Kou for your review. I will move the CF to the next
phase and see what happens.
Regards,
Yong
> On Nov 10, 2024, at 01:32, Rishu Bagga wrote:
>
> External Email
>
>> Thanks for pointing this out. Here is what I have tried:
>> 1. Manually build and install PostgreSQL from the latest source code.
>> 2. Following the instructions from src/bin/pg_upgrade to manually dump the
>> regression
>
> On Sep 12, 2024, at 13:39, Michael Paquier wrote:
>
> External Email
>
>
> I was looking at all that, and this is only moving code around. While
> the part for heap_xlog_logical_rewrite in rewriteheap.c is a bit sad
> but historical, the header cleanup in heapam.c is nice.
>
> Seeing heap
21 matches
Mail list logo
