]> granicus.if.org Git - postgresql/commit
Fix backup-block numbering in redo of b-tree split.
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Mon, 19 May 2014 10:16:24 +0000 (13:16 +0300)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Mon, 19 May 2014 10:28:04 +0000 (13:28 +0300)
commitc91a9b5a285e20e54cf90f3660ce51ce3a5c2ef4
tree7afa2a5b968b3be0c6576f2f2c94c9f0366db7c9
parenteb56812144bb310f7cb2f63e53d6d58f30df95b5
Fix backup-block numbering in redo of b-tree split.

I got the backup block numbers off-by-one in the commit that changed the
way incomplete-splits are handled. I blame the comments, which said
"backup block 1" and "backup block 2", even though the backup blocks
are numbered starting from 0, in the macros and functions used in replay.
Fix the comments and the code.

Per Jeff Janes' bug report about corruption caused by torn page writes.
The incorrect code is new in git master, but backpatch the comment change
down to 9.0, where the numbering in the redo-side macros  was changed.
src/backend/access/nbtree/nbtinsert.c
src/backend/access/nbtree/nbtxlog.c