Обсуждение: Removing a few more lseek() calls

Поиск
Список
Период
Сортировка

Removing a few more lseek() calls

От
Thomas Munro
Дата:
Hello,

Patch 0001 gets rid of the unconditional lseek() calls for SLRU I/O,
as a small follow-up to commit c24dcd0c.  Patch 0002 gets rid of a few
places that usually do a good job of avoiding lseek() calls while
reading and writing WAL, but it seems better to have no code at all.

-- 
Thomas Munro
https://enterprisedb.com
Вложения

Re: Removing a few more lseek() calls

От
Ashwin Agrawal
Дата:

On Sat, Mar 30, 2019 at 2:14 AM Thomas Munro <thomas.munro@gmail.com> wrote:
Hello,

Patch 0001 gets rid of the unconditional lseek() calls for SLRU I/O,
as a small follow-up to commit c24dcd0c.  Patch 0002 gets rid of a few
places that usually do a good job of avoiding lseek() calls while
reading and writing WAL, but it seems better to have no code at all.

I reviewed the changes and they look good to me. Code looks much cleaner after 2nd patch.
After these changes, only one usage of SLRU_SEEK_FAILED remains in SimpleLruDoesPhysicalPageExist().