From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
Cc: | Mario De Frutos Dieguez <mariodefrutos(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: signal handling in plpython |
Date: | 2016-10-14 17:50:35 |
Message-ID: | 12585.1476467435@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
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.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2016-10-14 17:53:24 | Re: signal handling in plpython |
Previous Message | Robert Haas | 2016-10-14 17:50:07 | Re: tablesample test failure with small TOAST_TUPLE_THRESHOLD |