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

From: Philip Alger <paalger0(at)gmail(dot)com>
To: Josef Šimánek <retro(at)ballgag(dot)cz>
Cc: jian he <jian(dot)universality(at)gmail(dot)com>, Jim Jones <jim(dot)jones(at)uni-muenster(dot)de>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Cary Huang <cary(dot)huang(at)highgo(dot)ca>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, josef(dot)simanek(at)gmail(dot)com
Subject: Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREATE TRIGGER statement
Date: 2025-11-03 01:18:44
Message-ID: CAPXBC8KQt=8Z_tbgzORgoOKXOYnuOpiKU2C7EWcjuvHUpoabAg@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Josef,

Would it make sense to rename trigger related variables to "trig"
> prefix instead of "trg" as is done in other functions in the same file
> (for example in function pg_get_triggerdef)?
>

Not sure it would be the same as triggerdef uses trig, while
triggerdef_worker uses a mix of trig and tg in the same function. It would
be another patch, I believe, to and correct a prefix, then you'd have to
normalize all the other functions within the same file.

--
Best,
Phil Alger

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Philip Alger 2025-11-03 01:22:26 Re: [PATCH] Add pg_get_trigger_ddl() to retrieve the CREATE TRIGGER statement
Previous Message Ranier Vilela 2025-11-03 00:34:11 Re: Avoid overflow (src/backend/utils/adt/formatting.c)