| From: | Michael Banck <mbanck(at)gmx(dot)net> |
|---|---|
| To: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
| Cc: | pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: add list of major features to the v19 release notes |
| Date: | 2026-07-02 11:15:39 |
| Message-ID: | 20260702111539.GA15075@p46.dedyn.io;lightning.p46.dedyn.io |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
yay for having this in place earlier this time.
Some comments:
On Wed, Jul 01, 2026 at 04:38:14PM -0500, Nathan Bossart wrote:
> + <listitem>
> + <para>
> + Logical replication now
> + <link linkend="logical-replication-sequences">replicates sequence values</link>,
> + and it can be enabled without a server restart when
> + <xref linkend="guc-wal-level"/> is set to <literal>replica</literal>.
> + </para>
> + </listitem>
Maybe it's because I'm not a native speaker, but I tripped over the
second part and parsed it as a function of the replicate sequence
values, not logical replication. Would it help readers to drop the "it",
i.e. write "and can be enabled [...]"?
> + <listitem>
> + <para>
> + Faster performance in many areas, including automatic scaling of the
> + number of <link linkend="guc-io-max-workers">I/O worker processes</link>,
> + quicker foreign-key checks, and further planning and execution
> + optimizations.
> + </para>
> + </listitem>
(This is kinda a catch-all statement and might go well as last item,
dunno).
> + <listitem>
> + <para>
> + Data checksums can now be
> + <link linkend="checksums-online-enable-disable">enabled or disabled while the database is running</link>.
> + </para>
> + </listitem>
I think we still don't say "the database" when we talk about a
server/instance/cluster. It should either be "the cluster" or "the
database server".
> + <listitem>
> + <para>
> + <xref linkend="sql-update"/> and <xref linkend="sql-delete"/> can now
> + change or remove data for just part of a time range via the new
> + <literal>FOR PORTION OF</literal> clause.
> + </para>
> + </listitem>
I think this sentence should include something about temporal
ranges/keys, or maybe just s/time range/temporal range/?
Michael
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Shlok Kyal | 2026-07-02 11:18:04 | Re: Proposal: Conflict log history table for Logical Replication |
| Previous Message | Andrei Lepikhov | 2026-07-02 11:05:57 | Re: RFC: Logging plan of the running query |