On Wed, Apr 05, 2023 at 04:07:21AM +0000, Imseih (AWS), Sami wrote:
>> - es_processed: number of tuples processed during one ExecutorRun()
>> call.
>> - es_total_processed: total number of tuples aggregated across all
>> ExecutorRun() calls.
>
> I thought hard about this point and for some reason I did not want to
> mention ExecutorRun in the comment. But, I agree with what you suggest.
> It's more clear as to the intention of the fields.
>
> Attached is v5 addressing the comments.
Thanks, this should be enough to persist the number of tuples tracked
across multiple ExecutorRun() calls. This looks pretty good to me.
We should do something about providing more control over that to
libpq in the long run, IMO, and have more test coverage, but let's see
about that in 17~.
--
Michael