On 04/11/2026 14:37, Fujii Masao wrote:
> Yeah, since AFAICT there seems no need to invoke registered callbacks,
> ResetExprContext() might be sufficient. Also, ResetPerTupleExprContext()
> would be better for consistency with CopyFrom(), which uses it?
>
IIUC es_per_tuple_exprcontext will always be non-NULL because of
GetPerTupleExprContext(), so in theory the extra check in
ResetPerTupleExprContext() is a bit redundant, but I agree it's a good
idea to keep the code consistent.
Thanks!
Best, Jim