Properly check index mark/restore in ExecSupportsMarkRestore.
authorAndrew Gierth <[email protected]>
Tue, 24 Nov 2020 20:58:32 +0000 (20:58 +0000)
committerAndrew Gierth <[email protected]>
Tue, 24 Nov 2020 21:58:32 +0000 (21:58 +0000)
commit660b89928d18386de7755565c008439ae75d1218
tree51e4bd2feba00e39a2e47dbd42b5cbe685177724
parentb0727ae99b7056ca88e9708c014a86f6e611d938
Properly check index mark/restore in ExecSupportsMarkRestore.

Previously this code assumed that all IndexScan nodes supported
mark/restore, which is not true since it depends on optional index AM
support functions. This could lead to errors about missing support
functions in rare edge cases of mergejoins with no sort keys, where an
unordered non-btree index scan was placed on the inner path without a
protecting Materialize node. (Normally, the fact that merge join
requires ordered input would avoid this error.)

Backpatch all the way since this bug is ancient.

Per report from Eugen Konkov on irc.

Discussion: https://postgr.es/m/[email protected]
src/backend/executor/execAmi.c
src/backend/optimizer/util/plancat.c
src/include/nodes/pathnodes.h