| 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: | Whole Thread | Raw Message | 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 |