Re: Applying XLR_INFO_MASK correctly when looking at WAL record information

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: Applying XLR_INFO_MASK correctly when looking at WAL record information
Дата
Msg-id CAB7nPqQ=b8V64JNRPAHpaHwmayNxxHXxiB1jqNxSONPrvqSDfA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Applying XLR_INFO_MASK correctly when looking at WAL record information  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
On Sat, Nov 5, 2016 at 2:29 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Michael Paquier <michael.paquier@gmail.com> writes:
>> I have just been trapped by the fact that in some code paths we look
>> at the RMGR information of a WAL record (record->xl_info) but the
>> filter dedicated to it, ~XLR_INFO_MASK, is not applied. This is
>> harmful now, but this could lead to problems if in the future new
>> record-level flags, similar to XLR_SPECIAL_REL_UPDATE, are added.
>> Disclaimer: I am looking at a patch where a record-level flag makes
>> sense, still this looks like a separate problem.
>
>> What about the patch attached to make things more consistent?
>
> Grepping found a couple of additional places that needed the same
> fix.

Ah, where wasShutdown is assigned. I thought I grepped it as well,
thanks for double-checking.

> Pushed with those additions.

Thanks.
-- 
Michael



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

Предыдущее
От: Andrew Dunstan
Дата:
Сообщение: Re: btree_gin and btree_gist for enums
Следующее
От: Peter Eisentraut
Дата:
Сообщение: Re: add more NLS to bin