On Wed, Apr 05, 2023 at 05:39:13PM -0400, Tom Lane wrote: > v5 seems OK to me except I think CreateExecutorState() should explicitly > zero the new es_total_processed field, alongside zeroing es_processed. > (I realize that the makeNode would have done it already, but our > coding conventions generally run towards not relying on that. This is > mainly for greppability, so you can find where a field is initialized.)
Makes sense to me. I'll look at that again today, potentially apply the fix on HEAD. -- Michael
signature.asc
Description: PGP signature