On Mon, Mar 13, 2017 at 8:51 PM, Michael Paquier <michael.paqu...@gmail.com> wrote: >> I agree with that, but I propose the attached version instead. It >> seems cleaner to have the entire test for setting BM_PERMANENT in one >> place rather than splitting it up as you did. > > Fine for me. You may want to update the comment of BM_PERMANENT in > buf_internals.h as Artur has mentioned upthread. For example by just > adding "and init forks".
OK, done, and back-patched all the way. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (firstname.lastname@example.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers