]> granicus.if.org Git - postgresql/commit
Fix CREATE MATVIEW/CREATE TABLE AS ... WITH NO DATA to not plan the query.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 27 Jun 2016 19:57:21 +0000 (15:57 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 27 Jun 2016 19:57:50 +0000 (15:57 -0400)
commit874fe3aea121b4ceb186d59ba6956e2f7d5aa0bb
tree4267a5cf5fbd3ee0fd59e9a7ad1b27b1b83d1157
parent6734a1cacd44f5b731933cbc93182b135b167d0c
Fix CREATE MATVIEW/CREATE TABLE AS ... WITH NO DATA to not plan the query.

Previously, these commands always planned the given query and went through
executor startup before deciding not to actually run the query if WITH NO
DATA is specified.  This behavior is problematic for pg_dump because it
may cause errors to be raised that we would rather not see before a
REFRESH MATERIALIZED VIEW command is issued.  See for example bug #13907
from Marian Krucina.  This change is not sufficient to fix that particular
bug, because we also need to tweak pg_dump to issue the REFRESH later,
but it's a necessary step on the way.

A user-visible side effect of doing things this way is that the returned
command tag for WITH NO DATA cases will now be "CREATE MATERIALIZED VIEW"
or "CREATE TABLE AS", not "SELECT 0".  We could preserve the old behavior
but it would take more code, and arguably that was just an implementation
artifact not intended behavior anyhow.

In 9.5 and HEAD, also get rid of the static variable CreateAsReladdr, which
was trouble waiting to happen; there is not any prohibition on nested
CREATE commands.

Back-patch to 9.3 where CREATE MATERIALIZED VIEW was introduced.

Michael Paquier and Tom Lane

Report: <20160202161407.2778.24659@wrigleys.postgresql.org>
src/backend/commands/createas.c
src/backend/commands/view.c
src/backend/nodes/makefuncs.c
src/include/nodes/makefuncs.h
src/test/regress/expected/matview.out
src/test/regress/expected/select_into.out
src/test/regress/sql/matview.sql
src/test/regress/sql/select_into.sql