Re: PATCH: Using BRIN indexes for sorted output

From: Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com>
To: Sergey Dudoladov <sergey(dot)dudoladov(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Cc: Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Greg Stark <stark(at)mit(dot)edu>
Subject: Re: PATCH: Using BRIN indexes for sorted output
Date: 2023-08-02 16:04:10
Message-ID: 9f7f5b1b-f59c-43b0-2b3b-b11e5213d157@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 8/2/23 17:25, Sergey Dudoladov wrote:
> Hello,
>
>> Parallel version is not supported, but I think it should be possible.
>
> @Tomas are you working on this ? If not, I would like to give it a try.
>

Feel free to try. Just keep it in a separate part/patch, to make it
easier to combine the work later.

>> static void
>> AssertCheckRanges(BrinSortState *node)
>> {
>> #ifdef USE_ASSERT_CHECKING
>>
>> #endif
>> }
>
> I guess it should not be empty at the ongoing development stage.
>
> Attached a small modification of the patch with a draft of the docs.
>

Thanks. FWIW it's generally better to always post the whole patch
series, otherwise the cfbot gets confused as it's unable to combine
stuff from different messages.

regards

--
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Nathan Bossart 2023-08-02 16:09:14 Re: add timing information to pg_upgrade
Previous Message Nathan Bossart 2023-08-02 15:59:15 Re: add timing information to pg_upgrade