Re: 2022-06-16 release announcement draft

From: "Jonathan S(dot) Katz" <jkatz(at)postgresql(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: 2022-06-16 release announcement draft
Date: 2022-06-14 01:15:14
Message-ID: 9ff6090c-1167-b928-2b18-1eb29927139c@postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 6/13/22 1:38 PM, Tom Lane wrote:
> "Jonathan S. Katz" <jkatz(at)postgresql(dot)org> writes:
>> Please review for technical accuracy and omissions.
>
> A few minor thoughts:
>
>> The PostgreSQL Global Development Group has released PostgreSQL 14.4 to fix an
>> issue that could cause silent data corruption when using the
>> [`CREATE INDEX CONCURRENTLY`](https://www.postgresql.org/docs/current/sql-createindex.html)
>> and [`REINDEX CONCURRENTLY`](https://www.postgresql.org/docs/current/sql-reindex.html)
>> commands.
>
> Maybe s/and/or/ ?

Fixed.

>> PostgreSQL 14.4 fixes an issue with the
>> [`CREATE INDEX CONCURRENTLY`](https://www.postgresql.org/docs/current/sql-createindex.html)
>> and [`REINDEX CONCURRENTLY`](https://www.postgresql.org/docs/current/sql-reindex.html)
>> that could cause silent data corruption of indexes.
>
> Either leave out "the" or add "commands". That is, "the FOO and BAR
> commands" reads fine, "the FOO and BAR" less so.

Yeah, that was likely an edit-o. Fixed.

> Also, I'm inclined
> to be a bit more specific and say that the problem is missing index
> entries, so maybe like "... fixes an issue that could cause the [CIC]
> and [RIC] commands to omit index entries for some rows".

Agreed. Edited in attached.

>> Once you upgrade your system to PostgreSQL 14.4, you can fix any silent data
>> corruption using `REINDEX CONCURRENTLY`.
>
> Perhaps it is also worth mentioning that you can use REINDEX without
> CONCURRENTLY, even before upgrading.

I'm hesitant on giving too many options. We did put out the "warning"
announcement providing this as an option. I do think that folks who are
running CIC/RIC are sensitive to locking, and a plain old "REINDEX" may
be viable except in an emergency.

>> * Report implicitly-created operator families (`CREATE OPERATOR CLASS`) to event
>> triggers.
>
> Maybe "(generated by `CREATE OPERATOR CLASS`)"? As-is, the parenthetical
> comment looks more like a mistake than anything else.

Fixed.

> The rest looks good.

Thanks for the review! Next version attached.

Jonathan

Attachment Content-Type Size
20220616updaterelease.md text/plain 5.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Geoghegan 2022-06-14 01:34:56 Re: 2022-06-16 release announcement draft
Previous Message Peter Geoghegan 2022-06-14 01:06:01 Re: Collation version tracking for macOS