Re: Cancelling parallel query leads to segfault

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: Cancelling parallel query leads to segfault
Date: 2018-01-29 14:51:00
Message-ID: CA+TgmoYhi-tGq60Dv8MQ3ewYsuL8-KvfOMKsQz=hrUZNVOFtVw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sat, Jan 27, 2018 at 10:45 PM, Andres Freund <andres(at)anarazel(dot)de> wrote:
> Hi Peter (and others who mucked around with related code),
>
> While testing another patch I found that cancelling a parallel query on
> master segfaults the leader in an interesting manner:
...
> It's clearly not OK to recurse back into the executing query after a
> failure, even less so if resources like the underlying DSM segment have
> already been freed.
>
> I suspect this is largely the fault of
>
> commit 8561e4840c81f7e345be2df170839846814fa004
> Author: Peter Eisentraut <peter_e(at)gmx(dot)net>
> Date: 2018-01-22 08:30:16 -0500
>
> Transaction control in PL procedures

Yeah, I think this must be the fault of that commit, and I think your
diagnosis is accurate.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Ryan Murphy 2018-01-29 14:51:53 Re: \describe*
Previous Message Robert Haas 2018-01-29 14:43:49 Re: [HACKERS] WIP: Aggregation push-down