Refactor cluster.c to use new routine get_index_isclustered()
authorMichael Paquier <michael@paquier.xyz>
Mon, 6 Apr 2020 02:44:23 +0000 (11:44 +0900)
committerMichael Paquier <michael@paquier.xyz>
Mon, 6 Apr 2020 02:44:23 +0000 (11:44 +0900)
commit8ef9451f58ee92d1cdb910e72010dbe75e76f9b8
tree0139670b79fed18fe65e33ec94dde4bd6f525f8b
parent33e05f89c53e5a1533d624046bb6fb0da7bb7141
Refactor cluster.c to use new routine get_index_isclustered()

This new cache lookup routine has been introduced in a40caf5, and more
code paths can directly use it.

Note that in cluster_rel(), the code was returning immediately if the
tuple's entry in pg_index for the clustered index was not valid.  This
commit changes the code so as a lookup error is raised instead,
something that could not happen from the start as we check for the
existence of the index beforehand, while holding an exclusive lock on
the parent table.

Author: Justin Pryzby
Reviewed-by: Álvaro Herrera, Michael Paquier
Discussion: https://postgr.es/m/20200202161718.GI13621@telsasoft.com
src/backend/commands/cluster.c