pgsql: Make concurrent refresh check early that there is a unique index

From: Fujii Masao <fujii(at)postgresql(dot)org>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Make concurrent refresh check early that there is a unique index
Date: 2016-02-15 17:16:51
Message-ID: E1aVMlX-0000Or-B1@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Make concurrent refresh check early that there is a unique index on matview.

In REFRESH MATERIALIZED VIEW command, CONCURRENTLY option is only
allowed if there is at least one unique index with no WHERE clause on
one or more columns of the matview. Previously, concurrent refresh
checked the existence of a unique index on the matview after filling
the data to new snapshot, i.e., after calling refresh_matview_datafill().
So, when there was no unique index, we could need to wait a long time
before we detected that and got the error. It was a waste of time.

To eliminate such wasting time, this commit changes concurrent refresh
so that it checks the existence of a unique index at the beginning of
the refresh operation, i.e., before starting any time-consuming jobs.
If CONCURRENTLY option is not allowed due to lack of a unique index,
concurrent refresh can immediately detect it and emit an error.

Author: Masahiko Sawada
Reviewed-by: Michael Paquier, Fujii Masao

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/31b6606c48edf7c008ffe91907c080404a8c8046

Modified Files
--------------
src/backend/commands/matview.c | 59 +++++++++++++++++++++++++++++++++++++-----
1 file changed, 53 insertions(+), 6 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Joe Conway 2016-02-15 21:22:28 pgsql: Correct Copyright year from 2015 to 2016
Previous Message Magnus Hagander 2016-02-15 10:44:11 pgsql: Fix typo