A couple of tiny performance hacks in _bt_step(). Remove PageIsEmpty
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 7 Dec 2005 18:03:48 +0000 (18:03 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 7 Dec 2005 18:03:48 +0000 (18:03 +0000)
commitf1b059af12673abf93ba13594799732bac83ed01
tree822946b5d6f997eb31c5de4fd4a8adf11cd7ff72
parent10a2df28c0dc51ac579ba2239d5f4c55e56c37d1
A couple of tiny performance hacks in _bt_step().  Remove PageIsEmpty
checks, which were once needed because PageGetMaxOffsetNumber would
fail on empty pages, but are now just redundant.  Also, don't set up
local variables that aren't needed in the fast path --- most of the
time, we only need to advance offnum and not step across a page boundary.
Motivated by noticing _bt_step at the top of OProfile profile for a
pgbench run.
src/backend/access/nbtree/nbtsearch.c