On Thu, Nov 3, 2016 at 9:57 AM, Haribabu Kommi <kommi.haribabu@gmail.com> wrote: > Thanks for your suggestion. Yes, I agree that adding a hint is good. > Updated patch is attached with addition of hint message. > > 2016-11-03 14:56:28.685 AEDT [7822] ERROR: cannot copy to view "ttt_v" > 2016-11-03 14:56:28.685 AEDT [7822] HINT: To enable copy to view, provide > an INSTEAD OF INSERT trigger > 2016-11-03 14:56:28.685 AEDT [7822] STATEMENT: COPY ttt_v FROM stdin;
Okay, Patch in general looks fine to me. One cosmetic comments, IMHO in PG we follow operator at end of the line, so move '&&' to end of the previous line.