Re: PL/R regression on windows, but not linux with master. - Mailing list pgsql-hackers

From Joe Conway
Subject Re: PL/R regression on windows, but not linux with master.
Date
Msg-id 3492e360-d198-b525-6ae7-05a8d098b050@joeconway.com
Whole thread Raw
In response to Re: PL/R regression on windows, but not linux with master.  (Dave Cramer <davecramer@gmail.com>)
Responses Re: PL/R regression on windows, but not linux with master.  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On 4/11/21 12:51 PM, Dave Cramer wrote:
> 
> 
> On Sun, 11 Apr 2021 at 12:43, Tom Lane <tgl@sss.pgh.pa.us 
> <mailto:tgl@sss.pgh.pa.us>> wrote:
> 
>     I wrote:
>      > Joe Conway <mail@joeconway.com <mailto:mail@joeconway.com>> writes:
>      >> Would an equivalent "PGWARNING" be something we are open to adding and
>      >> back-patching?
> 
>      > It's not real obvious how pl/r could solve this in a reliable way
>      > otherwise, so adding that would be OK with me, but I wonder whether
>      > back-patching is going to help you any.  You'd still need to compile
>      > against older headers I should think.  So I'd suggest
>      > (1) add PGWARNING in HEAD only
> 
>     Concretely, maybe like the attached?
> 
> 
> +1 from me.
> I especially like the changes to the comments as it's more apparent what they 
> should be used for.

+1

Looks great to me.

Joe

-- 
Crunchy Data - http://crunchydata.com
PostgreSQL Support for Secure Enterprises
Consulting, Training, & Open Source Development



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Possible SSI bug in heap_update
Next
From: Andrew Dunstan
Date:
Subject: Re: multi-install PostgresNode fails with older postgres versions