pgsql: Fix logging of pages skipped due to pins during vacuum. - Mailing list pgsql-committers

From Andres Freund
Subject pgsql: Fix logging of pages skipped due to pins during vacuum.
Date
Msg-id E1Y9C2K-0004Rv-83@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix logging of pages skipped due to pins during vacuum.

The new logging introduced in 35192f06 made the incorrect assumption
that scan_all vacuums would always wait for buffer pins; but they only
do so if the page actually needs to be frozen.

Fix that inaccuracy by removing the difference in log output based on
scan_all and just always remove the same message.  I chose to keep the
split log message from the original commit for now, it seems likely
that it'll be of use in the future.

Also merge the line about buffer pins in autovacuum's log output into
the existing "pages: ..." line. It seems odd to have a separate line
about pins, without the "topic: " prefix others have.

Also rename the new 'pinned_pages' variable to 'pinskipped_pages'
because it actually tracks the number of pages that could *not* be
pinned.

Discussion: 20150104005324.GC9626@awork2.anarazel.de

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/17eaae9897781c6dcc622b9327899329e4d2a4df

Modified Files
--------------
src/backend/commands/vacuumlazy.c |   32 ++++++++------------------------
1 file changed, 8 insertions(+), 24 deletions(-)


pgsql-committers by date:

Previous
From: Andres Freund
Date:
Subject: pgsql: Move comment about sun cc's __machine_rw_barrier being a full ba
Next
From: Andres Freund
Date:
Subject: pgsql: Protect against XLogReaderAllocate() failing to allocate memory.