| From: | "Li, Yong" <yoli(at)ebay(dot)com> |
|---|---|
| To: | "Bagga, Rishu" <bagrishu(at)amazon(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, "Debnath, Shawn" <sdn(at)ebay(dot)com> |
| Cc: | Andrey Borodin <x4mmm(at)yandex-team(dot)ru>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Aleksander Alekseev <aleksander(at)timescale(dot)com>, "Shyrabokau, Anton" <antons(at)ebay(dot)com> |
| Subject: | Re: Proposal to add page headers to SLRU pages |
| Date: | 2023-12-19 07:28:19 |
| Message-ID: | 628CFBEF-DF56-4D9A-A2F1-0A417848B6B6@ebay.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> 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 patch
and moved the on-disk upgrade logic to pg_upgrade/file.c.
I have also added this thread to the current Commitfest and hope this patch
will be part of the 17 release.
The commitfest link:
https://commitfest.postgresql.org/46/4709/
Regards,
Yong,
| Attachment | Content-Type | Size |
|---|---|---|
| slur_page_header_v2.patch | application/octet-stream | 53.6 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jeff Davis | 2023-12-19 07:32:30 | Re: Change GUC hashtable to use simplehash? |
| Previous Message | Amit Kapila | 2023-12-19 07:25:08 | Re: "pgoutput" options missing on documentation |