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

From: Manni Wood <manni(dot)wood(at)enterprisedb(dot)com>
To: Jim Jones <jim(dot)jones(at)uni-muenster(dot)de>
Cc: Nishant Sharma <nishant(dot)sharma(at)enterprisedb(dot)com>, Álvaro Herrera <alvherre(at)kurilemu(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-08 04:19:38
Message-ID: CAKWEB6ovQP+649hE9B=X+ZRbXR28C=3YwWwhD6a3aOBykYp8_A@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Nov 7, 2025 at 6:03 PM Jim Jones <jim(dot)jones(at)uni-muenster(dot)de> wrote:

>
>
> On 08/11/2025 00:38, Manni Wood wrote:
> > Alas, the build https://commitfest.postgresql.org/patch/6175/ <https://
> > commitfest.postgresql.org/patch/6175/> now fails, and I cannot reproduce
> > on my machine. Obviously there will be a v9...
>
> You forgot the declaration for build_tablespace_ddl_string[1]:
>
> ruleutils.c:13755:1: warning: no previous prototype for
> ‘build_tablespace_ddl_string’ [-Wmissing-prototypes]
> 13755 | build_tablespace_ddl_string(const Oid tspaceoid)
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> Best, Jim
>
> 1 - https://cirrus-ci.com/task/4855404196265984?logs=build#L1911
>
>
Thank you very much, Jim. Serves me right for looking at the error at the
end of the logs rather than the warning in the middle.

v9 is attached.
--
-- Manni Wood EDB: https://www.enterprisedb.com

Attachment Content-Type Size
v9-0002-Adds-pg_get_tablespace_ddl-function.patch text/x-patch 14.8 KB
v9-0001-Supporting-changes-for-pg_get_tablespace_ddl-func.patch text/x-patch 9.0 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Manni Wood 2025-11-08 04:46:28 Re: [PATCH] Add pg_get_tablespace_ddl() function to reconstruct CREATE TABLESPACE statement
Previous Message Bruce Momjian 2025-11-08 02:54:07 Re: Extra blank line in StrategyGetBuffer