Re: pg_dump: CREATE TABLE + CREATE RULE vs. relreplident

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  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + Everyone has their own god. +