Horton

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

Philip Alger <paalger0@gmail.com>

View thread
From: Philip Alger <paalger0@gmail.com>
To: Jim Jones <jim.jones@uni-muenster.de>
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-18T04:08:13Z
Lists: pgsql-hackers

Attachments

Hi Jim,

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

 Yes, you're correct. I've added that in v6 attached.

-- 
Best,
Phil Alger