Re: Propagate sanity checks of ProcessUtility() to standard_ProcessUtility()? - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Propagate sanity checks of ProcessUtility() to standard_ProcessUtility()?
Date
Msg-id ZeE35_P5dXCm7u_m@paquier.xyz
Whole thread Raw
In response to Re: Propagate sanity checks of ProcessUtility() to standard_ProcessUtility()?  (jian he <jian.universality@gmail.com>)
List pgsql-hackers
On Thu, Feb 29, 2024 at 04:10:26PM +0800, jian he wrote:
> why not just shovel these to standard_ProcessUtility.
> so ProcessUtility will looking consistent with (in format)
>  * ExecutorStart()
>  * ExecutorRun()
>  * ExecutorFinish()
>  * ExecutorEnd()

That's one of the points of the change: checking that only in
standard_ProcessUtility() may not be sufficient for utility hooks that
don't call standard_ProcessUtility(), so you'd stil want one in
ProcessUtility().
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Injection points: some tools to wait and wake
Next
From: "Zhijie Hou (Fujitsu)"
Date:
Subject: RE: Synchronizing slots from primary to standby