On Fri, Jun 2, 2017 at 12:51 AM, Amit Langote <langote_amit...@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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
