On 14/08/2020 16:51, Tom Lane wrote:
Heikki Linnakangas <hlinn...@iki.fi> writes:
Attached is a patch to fix this, as well as a similar issue in
pg_checksums. pg_basebackup and pgbench also print progres reports like
this, but they seem correct to me.

I wonder whether it'd be better to push the responsibility for this
into progress_report(), by adding an additional parameter "bool last"
or the like.  Then the callers would not need such an unseemly amount
of knowledge about what progress_report() is doing.

Good point. Pushed a patch along those lines.

- Heikki


Reply via email to