| From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
|---|---|
| To: | Christoph Berg <myon(at)debian(dot)org> |
| Cc: | Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org>, Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: plpgsql-trigger.html: Format TG_ variables as table (patch) |
| Date: | 2022-09-02 09:19:12 |
| Message-ID: | FB48C4E8-FE7F-4224-8257-C5EB53048FD7@yesql.se |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> On 1 Sep 2022, at 15:07, Christoph Berg <myon(at)debian(dot)org> wrote:
> Re: Dagfinn Ilmari Mannsåker
>>> I also shortened some "name of table" to just "table". Since the data
>>> type is "name", it's clear what "table" means.
>>
>> I think it reads better with the definite article and initial capital,
>> e.g. "The table that triggered ….".
>
> Since that's not a complete sentence anyway, I think "The" isn't
> necessary.
Looking at the docs for the other PLs there is quite a lot of variation on how
we spell this, fixing that inconsistency is for another patch though.
The patch missed to update the corresponding list for TG_ event trigger vars,
fixed in the attached.
--
Daniel Gustafsson https://vmware.com/
| Attachment | Content-Type | Size |
|---|---|---|
| v2-0001-doc-Use-more-concise-wording-for-pl-pgSQL-TG_-var.patch | application/octet-stream | 7.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dong Wook Lee | 2022-09-02 09:34:58 | Re: add test: pg_rowlocks extension |
| Previous Message | Drouvot, Bertrand | 2022-09-02 09:06:50 | Re: [PATCH] Query Jumbling for CALL and SET utility statements |