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
