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'. Italso 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/
OpenPGP_signature
Description: OpenPGP digital signature