pgsql: Simplify partitioned table creation vs. relcache

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Simplify partitioned table creation vs. relcache
Date: 2018-09-05 17:36:59
Message-ID: E1fxbjf-00016C-WA@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Simplify partitioned table creation vs. relcache

In the original code, we were storing the pg_inherits row for a
partitioned table too early: enough that we had a hack for relcache to
avoid falling flat on its face while reading such a partial entry. If
we finish the pg_class creation first and *then* store the pg_inherits
entry, we don't need that hack.

Also recognize that pg_class.relpartbound is not marked NOT NULL and
therefore it's entirely possible to read null values, so having only
Assert() protection isn't enough. Change those to if/elog tests
instead. This qualifies as a robustness fix, so backpatch to pg11.

In passing, remove one access that wasn't actually needed, and reword
one message to be like all the others that check for the same thing.

Reviewed-by: Amit Langote
Discussion: https://postgr.es/m/20180903213916.hh6wasnrdg6xv2ud@alvherre.pgsql

Branch
------
REL_11_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/52ab02d593e9d83c7bff8b6853aba59242ce967a

Modified Files
--------------
src/backend/commands/tablecmds.c | 10 +++-------
src/backend/partitioning/partbounds.c | 3 ++-
src/backend/utils/cache/partcache.c | 21 ++++-----------------
3 files changed, 9 insertions(+), 25 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Michael Paquier 2018-09-05 17:45:27 Re: pgsql: Clean up after TAP tests in oid2name and vacuumlo.
Previous Message Alvaro Herrera 2018-09-05 14:39:50 Re: pgsql: Clean up after TAP tests in oid2name and vacuumlo.