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

From Dave Cramer
Subject Re: PL/R regression on windows, but not linux with master.
Date
Msg-id CADK3HHKM_gF5XYjVEUyOkRNGca_YXAyaEzdFaW5RBrgXmCePXg@mail.gmail.com
Whole thread Raw
In response to Re: PL/R regression on windows, but not linux with master.  (Tomas Vondra <tomas.vondra@enterprisedb.com>)
List pgsql-hackers


On Sat, 10 Apr 2021 at 20:56, Tomas Vondra <tomas.vondra@enterprisedb.com> wrote:
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.


Thanks, I think ERROR is redefined in Windows as well for some strange reason.

Dave 

regards

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

pgsql-hackers by date:

Previous
From: Tomas Vondra
Date:
Subject: Re: PL/R regression on windows, but not linux with master.
Next
From: Justin Pryzby
Date:
Subject: Re: pgsql: autovacuum: handle analyze for partitioned tables