| From: | Kevin Grittner <kgrittn(at)gmail(dot)com> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>, David Fetter <david(at)fetter(dot)org>, Rajkumar Raghuwanshi <rajkumar(dot)raghuwanshi(at)enterprisedb(dot)com>, Amit Langote <amitlangote09(at)gmail(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>, pgsql-hackers-owner(at)postgresql(dot)org, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> |
| Subject: | Re: transition table behavior with inheritance appears broken (was: Declarative partitioning - another take) |
| Date: | 2017-05-01 16:10:52 |
| Message-ID: | CACjxUsM6jCgizYLJNHDAa2MGFXOQJQEmuca=+7bWmLJ5oh8-fA@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, May 1, 2017 at 10:01 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> It seems pretty clear to me that this is busted.
I don't think you actually tested anything that is dependent on any
of my patches there.
> Adding this as an open item. Kevin?
It will take some time to establish what legacy behavior is and how
the new transition tables are impacted. My first reaction is that a
trigger on the parent should fire for any related action on a child
(unless maybe the trigger is defined with an ONLY keyword???) using
the TupleDesc of the parent. Note that the SQL spec mandates that
even in a AFTER EACH ROW trigger the transition tables must
represent all rows affected by the STATEMENT. I think that this
should be independent of triggers fired at the row level. I think
the rules should be similar for updateable views.
This will take some time to investigate, discuss and produce a
patch. I think best case is Friday.
--
Kevin Grittner
VMware vCenter Server
https://www.vmware.com/
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Eisentraut | 2017-05-01 16:19:10 | pg_ctl wait exit code (was Re: [COMMITTERS] pgsql: Additional tests for subtransactions in recovery) |
| Previous Message | Andres Freund | 2017-05-01 16:09:18 | Re: snapbuild woes |