On Thu, Mar 12, 2015 at 03:14:48PM +0100, Andres Freund wrote: > Hi, > > On 2015-03-12 14:25:24 +0100, Marko Tiikkaja wrote: > > My colleague Per Lejontand brought to my attention that when dumping views > > with circular dependencies from a postgres version older than 9.4 using a > > recent pg_dump, the SQL looks something like the following: > > > > create table qwr(); > > create rule "_RETURN" as on select to qwr do instead select; > > > > In this case the relreplident column in pg_class for the view ends up being > > 'd', instead of the 'n' normally used for views. Patch to update > > relreplident when turning a table into a view is attached; this makes sure > > that the identity is NOTHING regardless of how the view was created. > > I think that's a good idea. > > > I consider this a bug fix, and suggest back patching to 9.4. > > I agree on backpatching it. Arguably we could additionally avoid > emitting the ALTER TABLE ... REPLICA IDENTITY for views that have > already been created with identity set like this. But I doubt it's worth > it.
Applied and backpatched to 9.4. -- Bruce Momjian <br...@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + Everyone has their own god. + -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers