Horton

Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREATE TRIGGER statement

Jim Jones <jim.jones@uni-muenster.de>

View thread
From: Jim Jones <jim.jones@uni-muenster.de>
To: Philip Alger <paalger0@gmail.com>
Cc: Andrew Dunstan <andrew@dunslane.net>, Cary Huang <cary.huang@highgo.ca>, jian he <jian.universality@gmail.com>, pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2025-10-17T23:44:52Z
Lists: pgsql-hackers 

On 10/17/25 21:07, Philip Alger wrote:
> The rationale behind it is here: https://www.postgresql.org/message-
> id/945db7c5-be75-45bf-b55b-cb1e56f2e3e9%40dunslane.net <https://
> www.postgresql.org/message-id/945db7c5-be75-45bf-b55b-
> cb1e56f2e3e9%40dunslane.net>

Ah, I wasn’t aware of this thread. That settles it then :)

Documentation also LGTM.

One last thing: should we perhaps check for NULL before calling
appendStringInfo here?

/* pg_get_triggerdef_worker retrieves the trigger definition */
res = pg_get_triggerdef_worker(trgOid, false);

appendStringInfo(&buf, "%s;", res);


Best, Jim