| 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 |
| 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 |