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 |
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 |