Re: Progress reporting for pg_verify_checksums
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Fabien COELHO <coelho@cri.ensmp.fr>
Cc: Michael Banck <michael.banck@credativ.de>, Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp>, alvherre@2ndquadrant.com, mailings@oopsware.de, thomas.munro@enterprisedb.com, pgsql-hackers@postgresql.org
Date: 2019-04-01T06:57:52Z
Lists: pgsql-hackers
On Sat, Mar 30, 2019 at 06:52:56PM +0100, Fabien COELHO wrote: > I do not think that it matters. I like to see things moving, and the > performance impact is null. Another point is that this bloats the logs redirected to a file by 4 compared to the initial proposal. I am not sure that this helps much for anybody. > I do not think that it is a good idea, because Michael is thinking of adding > some throttling capability, which would be a very good thing, but which will > need something precise, so better use the precise stuff from the start. > Also, the per second stuff induces rounding effects at the beginning. Let's revisit that when the need shows up then. I'd rather have us start with a basic set of metrics which can be extended later on. > Hmmm. I like this information because I this is where I have expectations, > whereas I'm not sure whether 1234 seconds for 12.3 GB is good or bad, but I > know that 10 MB/s on my SSD is not very good. Well, with some progress generated once per second you are one substraction away to guess how much has been computed in the last N second... -- Michael
Commits
-
Add progress reporting to pg_checksums
- 280e5f14056b 12.0 landed
-
Fix thinko when bumping on temporary directories in pg_verify_checksums
- da453004869d 11.3 landed
-
Fix thinko when bumping on temporary directories in pg_checksums
- 6eebfdc38b17 12.0 landed