]> granicus.if.org Git - postgresql/commit
Ignore whole-rows in INSERT/CONFLICT with partitioned tables
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Mon, 16 Apr 2018 18:50:57 +0000 (15:50 -0300)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Mon, 16 Apr 2018 18:52:28 +0000 (15:52 -0300)
commit158b7bc6d77948d2f474dc9f2777c87f81d1365a
tree8c8db23a6754aedb011b81689066904c9fa8cb87
parent3a2d6365984d2cd98ee03467cd901785941cf772
Ignore whole-rows in INSERT/CONFLICT with partitioned tables

We had an Assert() preventing whole-row expressions from being used in
the SET clause of INSERT ON CONFLICT, but it seems unnecessary, given
some tests, so remove it.  Add a new test to exercise the case.

Still at ExecInitPartitionInfo, we used map_partition_varattnos (which
constructs an attribute map, then calls map_variable_attnos) using
the same two relations many times in different expressions and with
different parameters.  Constructing the map over and over is a waste.
To avoid this repeated work, construct the map once, and use
map_variable_attnos() directly instead.

Author: Amit Langote, per comments by me (Álvaro)
Discussion: https://postgr.es/m/20180326142016.m4st5e34chrzrknk@alvherre.pgsql
src/backend/executor/execPartition.c
src/test/regress/expected/insert_conflict.out
src/test/regress/sql/insert_conflict.sql