Re: [PATCH] Add pg_get_event_trigger_ddl() function - Mailing list pgsql-hackers

From Zsolt Parragi
Subject Re: [PATCH] Add pg_get_event_trigger_ddl() function
Date
Msg-id CAN4CZFM+j04o_gejjYZVZKMYmF4N-6+4wOavvFxDR78FY6TnwQ@mail.gmail.com
Whole thread
In response to [PATCH] Add pg_get_event_trigger_ddl() function  (Ian Lawrence Barwick <barwick@gmail.com>)
List pgsql-hackers
> That did cross my mind, but while "enabled" feels more natural grammatically,
> "enable" corresponds to the "ENABLE" clause which is to be emitted (or omitted).
> Except when it's "DISABLE"... I can't think of any better phrasing
> which is reasonably
> short and conveys the meaning "emit the clause which indicates the trigger is to
> be disabled or enabled, and if the latter in what context".

Yeah, I agree with this reasoning, I wasn't even sure to mention this
or not. I only did because the ALTER EVENT TRIGGER documentation also
mentions "disabled", so there's some precedent of using that form.

Keeping it "enable" is fine with me.



pgsql-hackers by date:

Previous
From: shveta malik
Date:
Subject: Re: Support EXCEPT for ALL SEQUENCES publications
Next
From: Chao Li
Date:
Subject: Fix missing FORMAT when deparsing JSON_ARRAY(query)