Add test for HeapBitmapScan's broken skip_fetch optimization
authorAndres Freund <andres@anarazel.de>
Wed, 2 Apr 2025 18:58:39 +0000 (14:58 -0400)
committerAndres Freund <andres@anarazel.de>
Wed, 2 Apr 2025 18:58:39 +0000 (14:58 -0400)
commit24da5b239a4b50798fb2cbe46597cc932a5210f2
treef14f8293d84baaffa284fabfc16afafd2290a2a7
parent459e7bf8e2f8ab894dc613fa8555b74c4eef6969
Add test for HeapBitmapScan's broken skip_fetch optimization

In the previous commit HeapBitmapScan's skip_fetch optimization was removed,
due to being broken in not easily fixable ways. Add a test that verifies we
don't re-introduce this bug if somebody tries to re-add the feature.

Only add the test to master for now, it's possible it's not entirely
stable. That seems sufficient, as we're not going to re-introduce the feature
on the backbranches. I did verify that the test passes on all branches. If the
test turns out to be unproblematic, we can backpatch it later, should we feel
a need to do so.

Discussion: https://postgr.es/m/CAEze2Wg3gXXZTr6_rwC+s4-o2ZVFB5F985uUSgJTsECx6AmGcQ@mail.gmail.com
src/test/isolation/expected/index-only-bitmapscan.out [new file with mode: 0644]
src/test/isolation/isolation_schedule
src/test/isolation/specs/index-only-bitmapscan.spec [new file with mode: 0644]