Re: [COMMITTERS] 'pgsql/src/backend/parser gram.y parse_oper.c' - Mailing list pgsql-hackers

From Thomas G. Lockhart
Subject Re: [COMMITTERS] 'pgsql/src/backend/parser gram.y parse_oper.c'
Date
Msg-id 6b5d0709578c5b52843d25a56f662d3f
Whole thread Raw
List pgsql-hackers
> > Update of /usr/local/cvsroot/pgsql/src/backend/parser
> > In directory hub.org:/home/users/t/thomas/pgsql/src/backend/parser
> >
> > Modified Files:
> >       gram.y parse_oper.c
> > Log Message:
> > Change precedence for boolean operators to match expected behavior.
> > Change NOTICE/NOTICE/NOTICE/WARN elog messages to a single message
> >  with a few newline/tab breaks embedded in the string. Much cleaner I hope.
>
> I never liked the WARN message.  Basically, it is not a warning, but an
> error, and the statement is not executed.  Is this changed now, or
> should we change it.

I changed it to a single WARN. The type of error it is signalling is similar to
other errors which are flagged as "WARN". Is there another level of message you
are suggesting we use? FATAL closes the backend doesn't it?

                                                - Tom

------------------------------

pgsql-hackers by date:

Previous
From: "Sergey V. Lunegov"
Date:
Subject: [HACKERS] subsribe
Next
From: "Vadim B. Mikheev"
Date:
Subject: Re: [HACKERS] subselect