Re: pgsql: Make archiver process an auxiliary process.

From: Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>
To: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Fujii Masao <fujii(at)postgresql(dot)org>
Cc: pgsql-committers <pgsql-committers(at)lists(dot)postgresql(dot)org>
Subject: Re: pgsql: Make archiver process an auxiliary process.
Date: 2021-06-17 11:28:36
Message-ID: 743dda39-ab93-eca5-65e9-1e6eb33f1668@oss.nttdata.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On 2021/06/16 13:29, Fujii Masao wrote:
>
>
> On 2021/06/12 19:36, Thomas Munro wrote:
>> On Mon, Mar 15, 2021 at 5:14 PM Fujii Masao <fujii(at)postgresql(dot)org> wrote:
>>> Make archiver process an auxiliary process.
>>>
>>> This commit changes WAL archiver process so that it's treated as
>>> an auxiliary process and can use shared memory. This is an infrastructure
>>> patch required for upcoming shared-memory based stats collector patch
>>> series. These patch series basically need any processes including archiver
>>> that can report the statistics to access to shared memory. Since this patch
>>> itself is useful to simplify the code and when users monitor the status of
>>> archiver, it's committed separately in advance.
>>
>> I think this needs to handle ProcSignalBarrierPending.  Otherwise,
>> code that tries to use proc signal barriers (like CF #2962 and #2602)
>> might hang forever when an archiver is running.
>
> Thanks for the comment! I will investigate this.

Attached patch changes archiver so that it also handles barrier events.

Regards,

--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

Attachment Content-Type Size
archiver_handles_signal_barrier_v1.patch text/plain 2.2 KB

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Thomas Munro 2021-06-17 11:32:25 Re: pgsql: Make archiver process an auxiliary process.
Previous Message Peter Eisentraut 2021-06-17 07:05:07 pgsql: Fix subtransaction test for Python 3.10