On Fri, Feb 28, 2025 at 12:43 PM R, Rakshit <rakshi...@intel.com> wrote:

> > I don't think "another extension might use it someday" makes a very strong 
> > case,
> > particularly for something that requires a new dependency.
>
> The x86-simdsort library is an optional dependency in Postgres. Also the new 
> list sort implementation which uses the x86-simdsort library does not impact 
> any of the existing workflows in Postgres.

"Optional" and "Does not impact" are not great selling points to get
us to take a 1500 line patch. As we told you in November, list_sort
isn't critical for us. You need to start with the user and work
backwards to the technology. Don't pick a technology and try to sell
people on using it.

> We ran our extension to stress list sort with low cardinality inputs. For eg, 
> for an array of size 100k having repeated values in the range 1-10 we still 
> see a gain of around 20% in throughput.
> We will collect more data for low cardinality inputs and with AVX2 too.

Thanks for the news, those are encouraging results.

-- 
John Naylor
Amazon Web Services


Reply via email to