Refactor nbtree fastpath optimization.
authorPeter Geoghegan <pg@bowt.ie>
Wed, 18 Mar 2020 21:42:49 +0000 (14:42 -0700)
committerPeter Geoghegan <pg@bowt.ie>
Wed, 18 Mar 2020 21:42:49 +0000 (14:42 -0700)
commitb029395f5e616e0f2b1131b2c7ecb6640f30c055
tree5a8b6f1461a70b1a1947f1fc26939aecea696b7d
parenta2b1faa0f279f18517adc1dd969e5f026eb51d33
Refactor nbtree fastpath optimization.

Commit 2b272734, which added the fastpath rightmost leaf page cache
insert optimization, added code to _bt_doinsert() to handle using and
invalidating the backend local block cache.  It doesn't seem like a good
place to handle these low level details, though.  _bt_doinsert() is
supposed to be a high level function -- it is the main entry point to
nbtinsert.c.

Restructure the code by placing handling of the rightmost block cache at
the start of a new _bt_search() shim function, _bt_search_insert().  The
new function is called from _bt_doinsert(), which uses it as a
_bt_search() variant that conveniently accepts its BTInsertState state
as an argument.  _bt_doinsert() no longer needs to directly consider the
fastpath optimization.

Discussion: https://postgr.es/m/CAH2-Wzk59cxKJRd=rfbyub6-V4yWRjsOYRkUNHBLT1P1GdtCQQ@mail.gmail.com
src/backend/access/nbtree/nbtinsert.c