public inbox for [email protected]  

help / color / mirror / Atom feed

From: Jonathan S. Katz <[email protected]>
To: David Rowley <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Subject: Re: PostgreSQL 17 Beta 1 release announcement draft
Date: Sun, 19 May 2024 17:02:10 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAApHDvoUNLM3mnNU9Jt3qLKH-HEwgvF0ost3D4_t0R7c-+ipKg@mail.gmail.com>
References: <[email protected]>
	<CAApHDvoUNLM3mnNU9Jt3qLKH-HEwgvF0ost3D4_t0R7c-+ipKg@mail.gmail.com>

On 5/15/24 10:42 PM, David Rowley wrote:
> Thanks for writing that up. It's always exciting to see this each year.
> 
> On Thu, 16 May 2024 at 13:45, Jonathan S. Katz <[email protected]> wrote:
>> * Please indicate if you believe there's a notable omission, or if we
>> should omit any of these callouts
> 
> I'd say the streaming read stuff added in b5a9b18cd and subsequent
> commits like b7b0f3f27 and 041b96802 are worth a mention. I'd be happy
> to see this over the IS NOT NULL qual stuff that I worked on in there
> or even the AVX512 bit counting. Speeding up a backwater aggregate
> function is nice, but IMO, not compatible with reducing the number
> reads.
> There's some benchmarking in a youtube video:
> https://youtu.be/QAYzWAlxCYc?si=L0UT6Lrf067ZBv46&t=237

Nice! Definitely agree on including this - it wasn't initially clear to 
me on the read of the release notes. I'll update it. Please see in the 
next revision (will posted upthread), proposed text here for 
convenience, as I'm not sure I'm appropriately capturing it:

==
This release introduces adds an interface to stream I/O, and can show 
performance improvements when performing sequential scans and running 
[`ANALYZE`](https://www.postgresql.org/docs/17/sql-analyze.html).
==

The AVX-512 bit counting showed solid impact[1] on the binary distance 
functions in pgvector (I have to re-run again w/v17, as I seem to recall 
seeing some numbers that boosted it 5-7x [but recall isn't 100% ;)]).

>> * Please indicate if a description is confusing - I'm happy to rewrite
>> to ensure it's clearer.
>>
>> Please provide feedback no later than Wed 2024-05-22 18:00 UTC.
> 
> The only other thing I saw from a quick read was a stray "the" in "the
> copy proceed even if the there is an error inserting a row."

Thanks!

Jonathan

[1] https://github.com/pgvector/pgvector/pull/519



Attachments:

  [application/pgp-signature] OpenPGP_signature.asc (840B, ../[email protected]/2-OpenPGP_signature.asc)
  download

view thread (2+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: PostgreSQL 17 Beta 1 release announcement draft
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox