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 |