| From: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
|---|---|
| To: | Andrey Borodin <x4mmm(at)yandex-team(dot)ru> |
| Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Костя Кузнецов <chapaev28(at)ya(dot)ru> |
| Subject: | Re: GiST VACUUM |
| Date: | 2018-07-19 11:20:51 |
| Message-ID: | 77c7023c-18d4-8f04-08ed-a5c5633046ea@iki.fi |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 19/07/18 13:52, Andrey Borodin wrote:
> Hi!
>
>> 19 июля 2018 г., в 1:12, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>
>> написал(а):
>>
>> Yeah, please, I think this is the way to go.
>
> Here's v11 divided into proposed steps.
Thanks, one quick question:
> /* We should not unlock buffer if we are going to jump left */
> if (needScan)
> {
> GistBDItem *ptr = (GistBDItem *) palloc(sizeof(GistBDItem));
> ptr->buffer = buffer;
> ptr->next = bufferStack;
> bufferStack = ptr;
> }
> else
> UnlockReleaseBuffer(buffer);
Why? I don't see any need to keep the page locked, when we "jump left".
- Heikki
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Pavel Stehule | 2018-07-19 11:22:00 | Re: Runtime partition pruning for MergeAppend |
| Previous Message | Andrey Borodin | 2018-07-19 10:52:34 | Re: GiST VACUUM |