Confusing comment in tidbitmap.c

Поиск
Список
Период
Сортировка
От David Rowley
Тема Confusing comment in tidbitmap.c
Дата
Msg-id CAApHDvrYG5ooAMkxbiEs3fs_wkqk0ibfii=ghmtNJ9rTb_nVKg@mail.gmail.com
обсуждение исходный текст
Ответы Re: Confusing comment in tidbitmap.c  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
The following comment above #define PAGES_PER_CHUNK in tibbitmap.c appears to be incorrect:

"But we
 * also want PAGES_PER_CHUNK to be a power of 2 to avoid expensive integer
 * remainder operations.  So, define it like this:"

I don't quite follow this as it does nothing of the kind. Check tbm_page_is_lossy() where we do: bitno = pageno % PAGES_PER_CHUNK;

Or am I missing something about the compiler optimizing that to: bitno = pageno & 255; ?

Regards

David Rowley

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Amit Kapila
Дата:
Сообщение: Re: pg_basebackup vs. Windows and tablespaces
Следующее
От: David Rowley
Дата:
Сообщение: Re: speedup tidbitmap patch: cache page