Re: Progress reporting for pg_verify_checksums
Michael Banck <michael.banck@credativ.de>
From: Michael Banck <michael.banck@credativ.de>
To: Thomas Munro <thomas.munro@enterprisedb.com>
Cc: Bernd Helmle <bernd.helmle@credativ.de>, Fabien COELHO <coelho@cri.ensmp.fr>, Pg Hackers <pgsql-hackers@postgresql.org>, Alvaro Herrera <alvherre@2ndquadrant.com>
Date: 2018-12-21T10:09:11Z
Lists: pgsql-hackers
Attachments
- pg_verify_checksums_progress_V5.patch (text/x-diff) patch
Hi, On Wed, Oct 03, 2018 at 11:50:36AM +1300, Thomas Munro wrote: > On Sat, Sep 29, 2018 at 1:07 AM Michael Banck <michael.banck@credativ.de> wrote: > Windows doesn't like sigaction: > > https://ci.appveyor.com/project/postgresql-cfbot/postgresql/build/1.0.15189 Thanks for the report and sorry for taking so long to reply. > I'm not sure if we classify this as a "frontend" program. Should it > be using pqsignal() from src/port/pqsignal.c? Or perhaps just > sigaction as you have it (pqsignal.c says that we require sigaction on > all Unices), but #ifndef WIN32 around that stuff, since SIGUSR1 is > never going to work anyway. I've used pqsignal now, and disabled that feature on Windows. I've also updated one comment and added an additional regression test. V5 attached. Michael -- Michael Banck Projektleiter / Senior Berater Tel.: +49 2166 9901-171 Fax: +49 2166 9901-100 Email: michael.banck@credativ.de credativ GmbH, HRB Mönchengladbach 12080 USt-ID-Nummer: DE204566209 Trompeterallee 108, 41189 Mönchengladbach Geschäftsführung: Dr. Michael Meskes, Jörg Folz, Sascha Heuer Unser Umgang mit personenbezogenen Daten unterliegt folgenden Bestimmungen: https://www.credativ.de/datenschutz
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