git.postgresql.org Git - postgresql.git/commitdiff

git projects / postgresql.git / commitdiff

author Andres Freund <andres@anarazel.de>

2019年1月15日 19:59:32 +0000 (11:59 -0800)

committer Andres Freund <andres@anarazel.de>

2019年1月15日 20:04:32 +0000 (12:04 -0800)

The parent of some WCO expressions was, apparently by accident, set to
the the source of DML queries, rather than the target table. This
causes problems for the upcoming pluggable storage work, because the
target and source table might be of different storage types.

It's possible that this is already problematic, but neither
experimenting nor inquiries on -hackers have found them. So don't
backpatch for now.

Author: Andres Freund
Discussion: https://postgr.es/m/20181205225213.hiwa3kgoxeybqcqv@alap3.anarazel.de

src/backend/executor/nodeModifyTable.c patch | blob | blame | history

src/test/regress/expected/updatable_views.out patch | blob | blame | history

diff --git a/src/backend/executor/nodeModifyTable.c b/src/backend/executor/nodeModifyTable.c

index e18bc2a42e201496534964e46ba204868108922c..241711a81d773f2941e856519b14d71137bdec75 100644 (file)

--- a/src/backend/executor/nodeModifyTable.c

+++ b/src/backend/executor/nodeModifyTable.c

@@ -2270,7 +2270,7 @@ ExecInitModifyTable(ModifyTable *node, EState *estate, int eflags)

{

WithCheckOption *wco = (WithCheckOption *) lfirst(ll);

ExprState *wcoExpr = ExecInitQual((List *) wco->qual,

- mtstate->mt_plans[i]);

+ &mtstate->ps);

wcoExprs = lappend(wcoExprs, wcoExpr);

}

diff --git a/src/test/regress/expected/updatable_views.out b/src/test/regress/expected/updatable_views.out

index e64d693e9c6fe9c131722979f80d0bb255b4ae71..0ac08ca51d2672f78f4abbf768cf993c0ce70dbc 100644 (file)

--- a/src/test/regress/expected/updatable_views.out

+++ b/src/test/regress/expected/updatable_views.out

@@ -1782,31 +1782,31 @@ UPDATE rw_view1 SET a = a + 5; -- should fail

ERROR: new row violates check option for view "rw_view1"

DETAIL: Failing row contains (15).

EXPLAIN (costs off) INSERT INTO rw_view1 VALUES (5);

- QUERY PLAN

----------------------------------------------------------------

+ QUERY PLAN

+---------------------------------------------------------

Insert on base_tbl b

-> Result

- SubPlan 1

- -> Index Only Scan using ref_tbl_pkey on ref_tbl r

- Index Cond: (a = b.a)

- SubPlan 2

- -> Seq Scan on ref_tbl r_1

+ SubPlan 1

+ -> Index Only Scan using ref_tbl_pkey on ref_tbl r

+ Index Cond: (a = b.a)

+ SubPlan 2

+ -> Seq Scan on ref_tbl r_1

(7 rows)

EXPLAIN (costs off) UPDATE rw_view1 SET a = a + 5;

- QUERY PLAN

------------------------------------------------------------------

+ QUERY PLAN

+-----------------------------------------------------------

Update on base_tbl b

-> Hash Join

Hash Cond: (b.a = r.a)

-> Seq Scan on base_tbl b

-> Hash

-> Seq Scan on ref_tbl r

- SubPlan 1

- -> Index Only Scan using ref_tbl_pkey on ref_tbl r_1

- Index Cond: (a = b.a)

- SubPlan 2

- -> Seq Scan on ref_tbl r_2

+ SubPlan 1

+ -> Index Only Scan using ref_tbl_pkey on ref_tbl r_1

+ Index Cond: (a = b.a)

+ SubPlan 2

+ -> Seq Scan on ref_tbl r_2

(11 rows)

DROP TABLE base_tbl, ref_tbl CASCADE;

This is the main PostgreSQL git repository.

RSS Atom