On Tue, Feb 19, 2013 at 5:48 PM, Simon Riggs <si...@2ndquadrant.com> wrote: >>>> In access/transam/xlog.c we give the OS buffer caching a hint that we >>>> won't need a WAL file any time soon with >>>> >>>> posix_fadvise(openLogFile, 0, 0, POSIX_FADV_DONTNEED); >>>> > > I agree with Merlin and Joachim - if we have the call in one place, we > should have it in both.
You could argue that if it's considered beneficial in the case with no walsender, then you should definitely have it if there are walsenders around: The walsenders reopen and read those files which gives the OS reason to believe that other processes might do the same in the near future and hence that it should not evict those pages too early. Joachim -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers