Re: Yet another small patch - reorderbuffer.c:1099

Поиск
Список
Период
Сортировка
От Aleksander Alekseev
Тема Re: Yet another small patch - reorderbuffer.c:1099
Дата
Msg-id 20160405120740.5d92447c@fujitsu
обсуждение исходный текст
Ответ на Re: Yet another small patch - reorderbuffer.c:1099  (Michael Paquier <michael.paquier@gmail.com>)
Ответы Re: Yet another small patch - reorderbuffer.c:1099  (Andres Freund <andres@anarazel.de>)
Список pgsql-hackers
> I recall discussing this code with Andres, and I think that he has
> mentioned me this is intentional, because should things be changed for
> a reason or another in the future, we want to keep in mind that a list
> of TXIDs and a list of sub-TXIDs should be handled differently.

I see. If this it true I think there should be a comment that explains
it. When you read such a code you suspect a bug. Not mentioning that
static code analyzers (I'm currently experimenting with Clang and PVS
Studio) complain about code like this.

-- 
Best regards,
Aleksander Alekseev
http://eax.me/



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

Предыдущее
От: Magnus Hagander
Дата:
Сообщение: Re: Typo in src/interfaces/libpq/fe-connect.c
Следующее
От: Fujii Masao
Дата:
Сообщение: Re: Support for N synchronous standby servers - take 2