Re: Block level parallel vacuum WIP

From: David Steele <david(at)pgmasters(dot)net>
To: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
Cc: Claudio Freire <klaussfreire(at)gmail(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Block level parallel vacuum WIP
Date: 2017-03-03 14:01:22
Message-ID: 7bc0ca96-4198-d6aa-231a-85ca1ad4f80e@pgmasters.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 1/10/17 11:23 AM, Claudio Freire wrote:
> On Tue, Jan 10, 2017 at 6:42 AM, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
>>> Does this work negate the other work to allow VACUUM to use >
>>> 1GB memory?
>>
>> Partly yes. Because memory space for dead TIDs needs to be allocated
>> in DSM before vacuum worker launches, parallel lazy vacuum cannot use
>> such a variable amount of memory as that work does. But in
>> non-parallel lazy vacuum, that work would be effective. We might be
>> able to do similar thing using DSA but I'm not sure that is better.
>
> I think it would work well with DSA as well.
>
> Just instead of having a single segment list, you'd have one per worker.
>
> Since workers work on disjoint tid sets, that shouldn't pose a problem.
>
> The segment list can be joined together later rather efficiently
> (simple logical joining of the segment pointer arrays) for the index
> scan phases.

It's been a while since there was any movement on this patch and quite a
few issues have been raised.

Have you tried the approaches that Claudio suggested?

--
-David
david(at)pgmasters(dot)net

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tomas Vondra 2017-03-03 14:03:57 Re: PATCH: pageinspect / add page_checksum and bt_page_items(bytea)
Previous Message Karl O. Pinc 2017-03-03 13:51:52 Re: Patch to implement pg_current_logfile() function