Re: signal handling in plpython

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Mario De Frutos Dieguez <mariodefrutos(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: signal handling in plpython
Date: 2016-10-14 17:56:33
Message-ID: CA+TgmobUXFq22_zDdHN+C_oGeOxVjSbAOPSFVp=T+qqqf9O-6A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Oct 14, 2016 at 10:50 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Heikki Linnakangas <hlinnaka(at)iki(dot)fi> writes:
>> On 14 October 2016 16:22:12 EEST, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>>> I don't know whether to laugh or cry, but that code is a joke. Just
>>> silently fail if you can't get the lock?
>
>> Heh, ok, let me rephrase: I believe it's *intended* to be callable from a signal handler :). Whether it actually works is another question. Perhaps there's some mitigating conditions there, I don't know.
>
>> For our use case, it's actually not too bad if Py_AddPendingCall gives up and does nothing. Then the python function will simply not be interrupted until next SPI call, which is the current situation anyway.
>
> I dunno. If the failure were very low-probability, you could maybe live
> with that behavior, but I'm not sure it is. Presumably the Python
> interpreter loop is taking that lock once per statement (at least), so
> that it can tell if there's something to do. That'd suggest that the
> fraction of time in which the lock is held is not negligible.

I'm not sure that kibitzing the way the Python developers chose to
handle this is very helpful. Our job to use the APIs they've exposed,
not second-guess how they implemented them. The comment suggests that
the Python team thought that this would be reliable enough to be
acceptable, and I think we should assume they're right. If they're
wrong, they can fix that in a future release and our use of that API
will work that much better. Sitting on our hands gets us nowhere.

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2016-10-14 18:00:05 Re: Re: [COMMITTERS] pgsql: Extend framework from commit 53be0b1ad to report latch waits.
Previous Message Andres Freund 2016-10-14 17:53:24 Re: signal handling in plpython