Re: pg_waldump stucks with options --follow or -f and --stats or -z
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-11-16T02:56:01Z
Lists: pgsql-hackers
On Mon, Nov 15, 2021 at 07:32:38PM +0530, Bharath Rupireddy wrote: > pg_waldump options, --follow or -f(to keep polling once per second for > new WAL to appear) and --stats or -z don't work well together i.e. the > command stucks [1]. I think the pg_waldump should emit an error. Note > that the pg_basebakup does error out on incompatible options. > > Here's a small patch for fixing above along with a note in the documentation. > > Thoughts? I don't think that we should block this combination of options as you are proposing. The existing behavior is useful for users when it comes to an end position specified with -e, to be able to gather some stats on a cluster or an archive where we may not have all the contents wanted yet. > [1] The following commands stuck: > ./pg_waldump -p data/ -s 0/7000060 -f -z > ./pg_waldump -p data/ -s 0/7000060 -f --stats='record' > ./pg_waldump -p data/ -s 0/7000060 -f --stats='rmgr' Saying that, you are not completely wrong either, as following something while we won't print any stats at all is not really helpful. Another thing I can think of here is to make pg_waldump more responsive to the dump of the stats when interrupted, via XLogDumpDisplayStats(). -- Michael
Commits
-
pg_waldump: Emit stats summary when interrupted by SIGINT
- f2c52eeba919 15.0 landed