| From: | Dilip Kumar <dilipbalaut(at)gmail(dot)com> |
|---|---|
| To: | Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com> |
| Cc: | maxim(dot)boguk(at)gmail(dot)com, "pgsql-bugs(at)postgresql(dot)org" <pgsql-bugs(at)postgresql(dot)org> |
| Subject: | Re: BUG #14350: VIEW with INSTEAD OF INSERT TRIGGER and COPY. Missing feature or working as designed. |
| Date: | 2016-11-01 04:54:51 |
| Message-ID: | CAFiTN-tdd_DxpdpuHocWPCgxWnGeTBA9-i67hFNE4wmRHEnfrA@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs pgsql-hackers |
On Tue, Oct 4, 2016 at 1:07 PM, Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com> wrote:
> I think currently there is no handling of INSTEAD of triggers in the copy
> functionality.
>
> It didn't seem difficult to the support the same, until unless there are any
> problems for complext queries, so after adding the INSTEAD of triggers
> check and calling the ExecIRInsertTriggers function, the Copy is also
> working for the view.
>
> Attached is a POC patch of the same. I didn't checked all the possible
> scenarios.
We support insert into view in below 2 cases..
1. INSTEAD OF INSERT trigger
2. or an unconditional ON INSERT DO INSTEAD rule
In your patch we are supporting first one in COPY command, Will it not
be good to support second one also in COPY command ?
--
Regards,
Dilip Kumar
EnterpriseDB: http://www.enterprisedb.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Haribabu Kommi | 2016-11-01 07:10:10 | Re: [BUGS] BUG #14350: VIEW with INSTEAD OF INSERT TRIGGER and COPY. Missing feature or working as designed. |
| Previous Message | David G. Johnston | 2016-10-31 17:30:18 | Re: BUG #14397: Updatable VIEW doesn't work with ON CONFLICT from PgPreparedStatement |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Eisentraut | 2016-11-01 04:55:13 | DROP FUNCTION of multiple functions |
| Previous Message | Alvaro Herrera | 2016-11-01 04:43:31 | Re: Indirect indexes |