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

From Tomas Vondra
Subject Re: PL/R regression on windows, but not linux with master.
Date
Msg-id 6b4589ca-6476-17ff-9ecc-65205aa8345d@enterprisedb.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.
Re: PL/R regression on windows, but not linux with master.
List pgsql-hackers
On 4/11/21 2:38 AM, Dave Cramer wrote:
> 
> 
> 
> 
> On Sat, 10 Apr 2021 at 20:34, Tom Lane <tgl@sss.pgh.pa.us
> <mailto:tgl@sss.pgh.pa.us>> wrote:
> 
>     Dave Cramer <davecramer@gmail.com <mailto:davecramer@gmail.com>> writes:
>     > On Sat, 10 Apr 2021 at 20:24, Tom Lane <tgl@sss.pgh.pa.us
>     <mailto:tgl@sss.pgh.pa.us>> wrote:
>     >> That's quite bizarre.  What is the actual error level according to
>     >> the source code, and where is the error being thrown exactly?
> 
>     > Well it really is an ERROR, and is being downgraded on windows to
>     WARNING.
> 
>     That seems quite awful.
> 
>     However, now that I think about it, the elog.h error-level constants
>     were renumbered not so long ago.  Maybe you've failed to recompile
>     everything for v14?
> 
> 
> We see this on a CI with a fresh pull from master.
> 
> As I said I will dig into it and figure it out. 
> 

Well, plr.h does this:

#define WARNING        19
#define ERROR        20

which seems a bit weird, because elog.h does this (since 1f9158ba481):

#define WARNING        19
#define WARNING_CLIENT_ONLY    20
#define ERROR        21

Not sure why this would break Windows but not Linux, though.


regards

-- 
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



pgsql-hackers by date:

Previous
From: Dave Cramer
Date:
Subject: Re: PL/R regression on windows, but not linux with master.
Next
From: Dave Cramer
Date:
Subject: Re: PL/R regression on windows, but not linux with master.