• Alvaro Herrera's avatar
    Prevent drop of tablespaces used by partitioned relations · ebfe2dbd
    Alvaro Herrera authored
    When a tablespace is used in a partitioned relation (per commits
    ca410302 in pg12 for tables and 33e6c34c3267 in pg11 for indexes),
    it is possible to drop the tablespace, potentially causing various
    problems.  One such was reported in bug #16577, where a rewriting ALTER
    TABLE causes a server crash.
    
    Protect against this by using pg_shdepend to keep track of tablespaces
    when used for relations that don't keep physical files; we now abort a
    tablespace if we see that the tablespace is referenced from any
    partitioned relations.
    
    Backpatch this to 11, where this problem has been latent all along.  We
    don't try to create pg_shdepend entries for existing partitioned
    indexes/tables, but any ones that are modified going forward will be
    protected.
    
    Note slight behavior change: when trying to drop a tablespace that
    contains both regular tables as well as partitioned ones, you'd
    previously get ERRCODE_OBJECT_NOT_IN_PREREQUISITE_STATE and now you'll
    get ERRCODE_DEPENDENT_OBJECTS_STILL_EXIST.  Arguably, the latter is more
    correct.
    
    It is possible to add protecting pg_shdepend entries for existing
    tables/indexes, by doing
      ALTER TABLE ONLY some_partitioned_table SET TABLESPACE pg_default;
      ALTER TABLE ONLY some_partitioned_table SET TABLESPACE original_tablespace;
    for each partitioned table/index that is not in the database default
    tablespace.  Because these partitioned objects do not have storage, no
    file needs to be actually moved, so it shouldn't take more time than
    what's required to acquire locks.
    
    This query can be used to search for such relations:
    SELECT ... FROM pg_class WHERE relkind IN ('p', 'I') AND reltablespace <> 0
    Reported-by: default avatarAlexander Lakhin <exclusion@gmail.com>
    Discussion: https://postgr.es/m/16577-881633a9f9894fd5@postgresql.org
    Author: Álvaro Herrera <alvherre@alvh.no-ip.org>
    Reviewed-by: default avatarMichael Paquier <michael@paquier.xyz>
    ebfe2dbd
tablespace.source 12.3 KB