Re: [PATCH] Add pg_get_tablespace_ddl() function to reconstruct CREATE TABLESPACE statement

Поиск
Список
Период
Сортировка
От Manni Wood
Тема Re: [PATCH] Add pg_get_tablespace_ddl() function to reconstruct CREATE TABLESPACE statement
Дата
Msg-id CAKWEB6owLkUcohBzPv0dJp0qTO=7wCJbRPe_wQ7Y4-RSc0Fb0Q@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH] Add pg_get_tablespace_ddl() function to reconstruct CREATE TABLESPACE statement  (Álvaro Herrera <alvherre@kurilemu.de>)
Список pgsql-hackers


On Wed, Nov 12, 2025 at 10:15 AM Álvaro Herrera <alvherre@kurilemu.de> wrote:
On 2025-Nov-12, Manni Wood wrote:

> Thanks, Álvaro, for your continued help with this.
>
> I have attached v11 patches that use all of the fixes from your
> review.patch.txt.

OK, thanks, I pushed 0001 now.

I think you could claim that some routines currently in
src/backend/commands/tablespace.c logically belong in the new file, but
unless you want to take on the task of moving a lot of other routines
under commands/ to their respective catalog/ file, then I think it's
more or less fine as is.

To be clear, I do not intend to do anything with your 0002 patch [for
now].  I'm going to let Andrew take these DDL-producing functions in his
hands.  Here I'm just posting your 0002 again, to make the cfbot happy.

Thanks

--
Álvaro Herrera               48°01'N 7°57'E  —  https://www.EnterpriseDB.com/
"Nunca se desea ardientemente lo que solo se desea por razón" (F. Alexandre)

OK, thanks very much, Álvaro.

If you are OK with the current state of the patch, then I am happy to not move any more functions into their respective catalog/ files. My co-author, Nishant, should feel free to offer his opinion here too.

--
-- Manni Wood EDB: https://www.enterprisedb.com

В списке pgsql-hackers по дате отправления: