]> granicus.if.org Git - postgresql/commit
Fix transient mdsync() errors of truncated relations due to 72a98a6395.
authorAndres Freund <andres@anarazel.de>
Wed, 4 May 2016 08:54:20 +0000 (01:54 -0700)
committerAndres Freund <andres@anarazel.de>
Wed, 4 May 2016 08:54:20 +0000 (01:54 -0700)
commita712487087c73eee880ff1a7c50528cbab2f1b90
tree4d423e318a056ced0de24e6643619a8519c8adc4
parent613fb29a384c8a75146af7cfa433cfa61716f117
Fix transient mdsync() errors of truncated relations due to 72a98a6395.

Unfortunately the segment size checks from 72a98a6395 had the negative
side-effect of breaking a corner case in mdsync(): When processing a
fsync request for a truncated away segment mdsync() could fail with
"could not fsync file" (if previous segment < RELSEG_SIZE) because
_mdfd_getseg() now wouldn't return the relevant segment anymore.

The cleanest fix seems to be to allow the caller of _mdfd_getseg() to
specify whether checks for RELSEG_SIZE are performed. To allow doing so,
change the ExtensionBehavior enum into a bitmask. Besides allowing for
the addition of EXTENSION_DONT_CHECK_SIZE, this makes for a nicer
implementation of EXTENSION_REALLY_RETURN_NULL.

Besides mdsync() the only callsite that should change behaviour due to
this is mdprefetch() which now doesn't create segments anymore, even in
recovery. Given the uses of mdprefetch() that seems better.

Reported-By: Thom Brown
Discussion: CAA-aLv72QazLvPdKZYpVn4a_Eh+i4_cxuB03k+iCuZM_xjc+6Q@mail.gmail.com
src/backend/storage/smgr/md.c