Re: PL/R Median Busts Commit (Postgres 9.1.6 + plr 8.3.0.13 on Ubuntu 12.10 64 bit)

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Mark Kirkwood <mark(dot)kirkwood(at)catalyst(dot)net(dot)nz>
Cc: pgsql-bugs(at)postgresql(dot)org, Joe Conway <mail(at)joeconway(dot)com>
Subject: Re: PL/R Median Busts Commit (Postgres 9.1.6 + plr 8.3.0.13 on Ubuntu 12.10 64 bit)
Date: 2013-01-25 00:49:20
Message-ID: 29635.1359074960@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

Mark Kirkwood <mark(dot)kirkwood(at)catalyst(dot)net(dot)nz> writes:
> On 25/01/13 13:06, Tom Lane wrote:
>> Unless libR can be coerced into not screwing up our signal handlers,
>> I'd say that PL/R is broken beyond repair. That would be unfortunate.

> It looks like Joe has run into something similar with libR stealing
> SIGINT, he reinstalls it. A simple patch along the same lines for
> SIGUSR1 (attached) seems to fix the issue.

This certainly is not good enough, for either signal. What happens if
the signal arrives while libR still has control? These things being
asynchronous with respect to the receiving backend, we certainly can't
assume that that won't happen.

Why does libR think it should be messing with these handlers in the
first place?

regards, tom lane

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Mark Kirkwood 2013-01-25 00:56:41 Re: PL/R Median Busts Commit (Postgres 9.1.6 + plr 8.3.0.13 on Ubuntu 12.10 64 bit)
Previous Message Mark Kirkwood 2013-01-25 00:39:27 Re: PL/R Median Busts Commit (Postgres 9.1.6 + plr 8.3.0.13 on Ubuntu 12.10 64 bit)