Re: Making pgfdw_report_error statically analyzable

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Álvaro Herrera <alvherre(at)kurilemu(dot)de>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org, Robert Haas <robertmhaas(at)gmail(dot)com>
Subject: Re: Making pgfdw_report_error statically analyzable
Date: 2025-07-28 21:30:17
Message-ID: 768771.1753738217@sss.pgh.pa.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

=?utf-8?Q?=C3=81lvaro?= Herrera <alvherre(at)kurilemu(dot)de> writes:
> Hmm, what about 2c. having pgfdw_report_error() with hardcoded elevel,
> but complement it with pgfdw_report() that takes the elevel as argument,
> asserting that it's less than ERROR? Then the calls look like
> pgfdw_report(WARNING, "doo dee");

> which makes sense IMO and we don't have to worry about the future.

This is the same as my 2a except for the choice of function name.
I'd be fine with it, but Robert didn't like 2a.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message David Rowley 2025-07-28 21:30:41 Re: A performance regression issue with Memoize
Previous Message Álvaro Herrera 2025-07-28 21:25:58 Re: Making pgfdw_report_error statically analyzable