On Fri, Jun 2, 2017 at 12:51 AM, Amit Langote
<Langote_Amit_f8@lab.ntt.co.jp> wrote:
> Attached patch makes InitResultRelInfo() *always* initialize the
> partition's constraint, that is, regardless of whether insert/copy is
> through the parent or directly on the partition. I'm wondering if
> ExecInsert() and CopyFrom() should check if it actually needs to execute
> the constraint? I mean it's needed if there exists BR insert triggers
> which may change the row, but not otherwise. The patch currently does not
> implement that check.
I think it should. I mean, otherwise we're leaving a
probably-material amount of performance on the table.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company