On 5/9/22 9:43 AM, Euler Taveira wrote:
On Sun, May 8, 2022, at 5:51 PM, Jonathan S. Katz wrote:
Hi,

Please see attached draft for the 2022-05-12 release announcement.
LGTM.

Please provide feedback on accuracy and if there are any notable omissions.
* Several fixes for `contrib/pageinspect` to improve overall stability.
* Disable batch insertion in `contrib/postgres_fdw` when
`BEFORE INSERT ... FOR EACH ROW` triggers exist on the foreign table.

Should you omit 'contrib'? You mentioned ltree but don't say 'contrib/ltree'. It
also isn't used in a previous release announcement (see postgres_fdw).


[1] https://www.postgresql.org/about/news/postgresql-142-136-1210-1115-and-1020-released-2402/ <https://www.postgresql.org/about/news/postgresql-142-136-1210-1115-and-1020-released-2402/>

Hm, it looks like this is inconsistent with other announcements, as those have the "contrib" prefix[1][2].

I think in this specific case, `ltree` is the data type found in the `contrib/ltree` extension.

You raise a good point, which is in the release announcement, should we prefix contrib modules with "contrib"? Perhaps instead of shorthand we spell it out, e.g. "Several fixes for the `pageinspect` module..."

Jonathan

[1] https://www.postgresql.org/about/news/postgresql-134-128-1113-1018-9623-and-14-beta-3-released-2277/ [2] https://www.postgresql.org/about/news/postgresql-132-126-1111-1016-9621-and-9525-released-2165/

Attachment: OpenPGP_signature
Description: OpenPGP digital signature

Reply via email to