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
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Philip Alger
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Andrew Dunstan
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Philip Alger
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Jim Jones
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Philip Alger
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Philip Alger
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Jim Jones
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Philip Alger
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Jim Jones
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Philip Alger
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Jim Jones
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Philip Alger
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Jim Jones
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Marcos Pegoraro
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Philip Alger
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Marcos Pegoraro
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Philip Alger
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Philip Alger
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Philip Alger
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Tom Lane
- Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREAT... Philip Alger
