On Mon, Jan 2, 2023 at 5:23 AM Peter Eisentraut <peter.eisentr...@enterprisedb.com> wrote: > > On 30.11.22 23:34, Melanie Plageman wrote: > > I have attached a patchset with only the code changes contained in the > > previous patch 0003. I have broken the refactoring down into many > > smaller pieces for ease of review. > > To keep this moving along a bit, I have committed your 0002, which I > think is a nice little improvement on its own.
Thanks! I've attached a rebased patchset - v4. I also changed heapgettup_no_movement() to noinline (instead of inline). David Rowley pointed out that this might make more sense given how comparatively rare no movement scans are. - Melanie
v4-0004-Add-heapgettup-helpers.patch
Description: Binary data
v4-0002-Add-heapgettup_initial_page-helper.patch
Description: Binary data
v4-0005-Add-heapgettup_advance_page.patch
Description: Binary data
v4-0006-Refactor-heapgettup-and-heapgettup_pagemode.patch
Description: Binary data
v4-0003-Streamline-heapgettup-for-refactor.patch
Description: Binary data
v4-0001-Add-no-movement-scan-helper.patch
Description: Binary data
