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! -- Álvaro Herrera http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers