| From: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
|---|---|
| To: | Nikolay Shaplov <dhyan(at)nataraj(dot)su> |
| Cc: | Yura Sokolov <y(dot)sokolov(at)postgrespro(dot)ru>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Michael Paquier <michael(at)paquier(dot)xyz> |
| Subject: | Re: [PATCH] New [relation] option engine |
| Date: | 2023-12-08 12:59:09 |
| Message-ID: | 202312081259.2pwkb25emowf@alvherre.pgsql |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 2023-Dec-08, Nikolay Shaplov wrote:
> Theoretically I can create patch with full options.c as it is in the patch
> now, and use that code only in index AM, and keep reloption.c mostly
> unchanged.
>
> This will be total mess with two different options mechanisms working in the
> same time, but this might be much more easy to review. When we are done with
> the first step, we can change the rest.
> If this will help to finally include patch into postgres, I can do it. Will
> that help you to review?
I don't think that's better, because we could create slight
inconsistencies between the code used for index AMs and the users of
reloptions.
I'm not seeing any reasonable way to split this patch in smaller ones.
--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/
"No necesitamos banderas
No reconocemos fronteras" (Jorge González)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Daniel Gustafsson | 2023-12-08 12:59:22 | Re: initdb caching during tests |
| Previous Message | Alvaro Herrera | 2023-12-08 12:55:28 | Re: GUC names in messages |