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

From: Manni Wood <manni(dot)wood(at)enterprisedb(dot)com>
To: Nishant Sharma <nishant(dot)sharma(at)enterprisedb(dot)com>
Cc: Álvaro Herrera <alvherre(at)kurilemu(dot)de>, Jim Jones <jim(dot)jones(at)uni-muenster(dot)de>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCH] Add pg_get_tablespace_ddl() function to reconstruct CREATE TABLESPACE statement
Date: 2025-11-07 01:27:29
Message-ID: CAKWEB6pq+P-g6i7C4ZSZsjVqdcH7z4OeWf+xVX6qSK=tHw5p_A@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Nov 6, 2025 at 4:08 PM Manni Wood <manni(dot)wood(at)enterprisedb(dot)com>
wrote:

>
>
> On Wed, Nov 5, 2025 at 5:54 AM Nishant Sharma <
> nishant(dot)sharma(at)enterprisedb(dot)com> wrote:
>
>> Thanks Álvaro for the review comments on v4!
>>
>> PFA, v5 patch set. I have included all your review comments.
>>
>>
>> Regards,
>> Nishant Sharma.
>> EDB, Pune.
>> https://www.enterprisedb.com
>>
>
> The BSD build was failing with the error 'WARNING: tablespaces created by
> regression test cases should have names starting with "regress_"', so the
> attached patches should fix that.
>
> The windows build is also failing, on this error
> "../src/port/strerror.c(311): fatal error C1051: program database file,
> 'C:\cirrus\build\src\port\libpgport_srv.pdb', has an obsolete format,
> delete it and recompile", which I don't think is related to our patch.
> --
> -- Manni Wood EDB: https://www.enterprisedb.com
>

And once again, I am foiled by whitespace. Attached v7 fixes problems in
tests due to whitespace.
--
-- Manni Wood EDB: https://www.enterprisedb.com

Attachment Content-Type Size
v7-0002-Adds-pg_get_tablespace_ddl-function.patch text/x-patch 15.0 KB
v7-0001-Supporting-changes-for-pg_get_tablespace_ddl-func.patch text/x-patch 6.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bryan Green 2025-11-07 01:35:29 Re: [PATCH] Fix socket handle inheritance on Windows
Previous Message Bryan Green 2025-11-07 01:17:10 Re: [Patch] Windows relation extension failure at 2GB and 4GB