]> granicus.if.org Git - postgresql/commit
Allow SetHintBits() to succeed if the buffer's LSN is new enough.
authorAndres Freund <andres@anarazel.de>
Mon, 15 Feb 2016 21:15:35 +0000 (22:15 +0100)
committerAndres Freund <andres@anarazel.de>
Mon, 15 Feb 2016 21:48:51 +0000 (22:48 +0100)
commitdb76b1efbbab2441428a9ef21f7ac9ba43c52482
tree2be35f36c72ec21cbc1d037c80bcf6e4eabb7cec
parentcfafd8beadcd6f8c8a570ab2504a75c33c58e504
Allow SetHintBits() to succeed if the buffer's LSN is new enough.

Previously we only allowed SetHintBits() to succeed if the commit LSN of
the last transaction touching the page has already been flushed to
disk. We can't generally change the LSN of the page, because we don't
necessarily have the required locks on the page. But the required LSN
interlock does not mean the commit record has to be flushed immediately,
it just requires that the commit record will be flushed before the page is
written out. Therefore if the buffer LSN is newer than the commit LSN,
the hint bit can be safely set.

In a number of scenarios (e.g. pgbench) this noticeably increases the
number of hint bits are set. But more importantly it also keeps the
success rate up when flushing WAL less frequently. That was the original
reason for commit 4de82f7d7, which has negative performance consequences
in a number of scenarios. This will allow a followup commit to reduce
the flush rate.

Discussion: 20160118163908.GW10941@awork2.anarazel.de
src/backend/utils/time/tqual.c