On 9/19/25 12:50 PM, Nathan Bossart wrote:
On Fri, Sep 19, 2025 at 09:25:52AM -0500, Nathan Bossart wrote:
I'm hoping to commit this around 20:00 UTC today, and I will be happy to
address any feedback that folks have in the meantime.

Here's a v4.  The content is the same except for a typo fix, some
formatting adjustments that don't change anything in the generated page,
and a real commit message.  I'm okay with the content, but I figured I'd
note some thoughts:

+      An asynchronous I/O subsystem (AIO) that can improve performance of
+      sequential scans, bitmap heap scans, vacuums, and other operations.

I wondered whether we should put "(AIO)" before "subsystem", but I think
putting it next to "I/O" makes the line too busy.  Also, are there "other
operations", or is the rest of the list complete?

Will throw out the "we can just remove it" option like further down in the release notes, but figure we've put the AIO terminology out there enough it may be good to assign the too. Anyway, if we keep it, I'd suggest "asynchronous I/O (AIO)" given it's modifying that.

For "other operations", the release notes have "etc." in them. But if we want to hedge, we can do:

...that can improve performance of operations, including sequential scans, bitmap heap scans, and vacuums."

+      <link linkend="pgupgrade"><application>pg_upgrade</application></link>
+      now maintains optimizer statistics through upgrade.

I think "retain" might be a better verb than "maintain", but the meaning
seems clear either way.  Also, while we could probably omit "through
upgrade", the small amount of redundancy does (IMHO) reinforce the meaning
a bit.

OK with "retains", and OK w/dropping "through upgrade".

+      Support for "skip scan" lookups that allow
+      <link linkend="indexes-multicolumn">multicolumn B-tree indexes</link> to
+      be used in more cases.

Passive voice.  Perhaps this should be "that allow using ... in more
cases."

Agreed switching to active voice.

+      <link linkend="sql-createtable-parms-generated-stored">generated 
columns</link>
+      that compute their values during read operations.  This is now the
+      default for generated columns.

I liked the phrase "just-in-time" for this because it expresses how it
works.  Perhaps we should squeeze that in before "during read operations".

I think "during" and "just-in-time" are equivalent here. Also wanted to be sensitive to the fact we already have a feature called "just-in-time"/"JIT" for compilation, and didn't want people to confuse the two.

Jonathan

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature

Reply via email to