Re: Reducing WaitEventSet syscall churn

From: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Daniel Gustafsson <daniel(at)yesql(dot)se>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Reducing WaitEventSet syscall churn
Date: 2021-01-05 05:10:09
Message-ID: CA+hUKGKE=cKUChUjJqkgZUubmfBB=X8Fp+t=EDeVycLPNSEx7Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Jul 31, 2020 at 9:42 AM Thomas Munro <thomas(dot)munro(at)gmail(dot)com> wrote:
> On Thu, Jul 30, 2020 at 5:50 PM Thomas Munro <thomas(dot)munro(at)gmail(dot)com> wrote:
> > I pushed those three patches, but will wait for more discussion on the rest.
>
> And here's a rebase, to make cfbot happy.

And again.

To restate the two goals of the remaining patches:

1. Use FeBeWaitSet for walsender, instead of setting up and tearing
down temporary WaitEventSets all the time.
2. Use the standard WL_EXIT_ON_PM_DEATH flag for FeBeWaitSet, instead
of the special case ereport(FATAL, ... "terminating connection due to
unexpected postmaster exit" ...).

For point 2, the question I am raising is: why should users get a
special FATAL message in some places and not others, for PM death?
However, if people are attached to that behaviour, we could still
achieve goal 1 without goal 2 by handling PM death explicitly in
walsender.c and I'd be happy to post an alternative patch set like
that.

Attachment Content-Type Size
v7-0001-Use-WL_EXIT_ON_PM_DEATH-in-FeBeWaitSet.patch application/x-patch 3.3 KB
v7-0002-Introduce-symbolic-names-for-FeBeWaitSet-position.patch application/x-patch 4.0 KB
v7-0003-Use-FeBeWaitSet-for-walsender.c.patch application/x-patch 3.5 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Shinya11.Kato 2021-01-05 06:02:02 RE: [PATCH] Feature improvement for CLOSE, FETCH, MOVE tab completion
Previous Message vignesh C 2021-01-05 04:38:15 Re: Parallel Inserts in CREATE TABLE AS