On 2023-Sep-27, Amit Langote wrote: > Maybe the following is better: > > + /* > + * For expression nodes that support soft errors. Should be set to NULL > + * before calling ExecInitExprRec() if the caller wants errors thrown. > + */ > > ...as in the attached.
That's good. > Alvaro, do you think your concern regarding escontext not being in the > right spot in the ExprState struct is addressed? It doesn't seem very > critical to me to place it in the struct's 1st cacheline, because > escontext is not accessed in performance critical paths such as during > expression evaluation, especially with the latest version. (It would > get accessed during evaluation with previous versions.) > > If so, I'd like to move ahead with committing it. Yeah, looks OK to me in v21. -- Álvaro Herrera PostgreSQL Developer — https://www.EnterpriseDB.com/