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

Attachment: v6-0001-Add-pg_get_trigger_ddl-function.patch
Description: Binary data

Reply via email to