On Thu, Oct 31, 2013 at 10:22 PM, Kevin Grittner <kgri...@ymail.com> wrote: > "t.katsumata1...@gmail.com" <t.katsumata1...@gmail.com> wrote: > If there are no objections I'll apply this within a few days. I am not sure that adding a boolean flag introducing a concept related to matview inside checkRuleResultList is the best approach to solve that. checkRuleResultList is something related only to rules, and has nothing related to matviews in it yet. I have been pondering myself about this issue, and wouldn't it be better to directly enforce targetList in checkRuleResultList call at an upper level with the column name list given by users if there is any instead of the list of columns of the SELECT query?
After looking closely at the code this looks to be a better approach from a general viewpoint. I didn't got the time to write a patch but this is the conclusion I reached after some analysis at least... -- Michael -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers