On Sat, Nov 15, 2014 at 2:23 AM, Alvaro Herrera <alvhe...@2ndquadrant.com> wrote: > > Michael Paquier wrote: > > > Btw, perhaps this diff should be pushed as a different patch as this is a > > rather different thing: > > - if (heapRelation->rd_rel->relpersistence == RELPERSISTENCE_UNLOGGED > > && > > + if (indexRelation->rd_rel->relpersistence == > > RELPERSISTENCE_UNLOGGED && > > !smgrexists(indexRelation->rd_smgr, INIT_FORKNUM) > > As this is a correctness fix, it does not seem necessary to back-patch it: > > the parent relation always has the same persistence as its indexes. > > There was an argument for doing it this way that only applies if this > patch went in, but I can't remember now what it was. > > Anyway I pushed the patch after some slight additional changes. Thanks! >
Thanks! -- Fabrízio de Royes Mello Consultoria/Coaching PostgreSQL >> Timbira: http://www.timbira.com.br >> Blog: http://fabriziomello.github.io >> Linkedin: http://br.linkedin.com/in/fabriziomello >> Twitter: http://twitter.com/fabriziomello >> Github: http://github.com/fabriziomello
