[MASSMAIL]call ATPostAlterTypeParse inconsistency

Поиск
Список
Период
Сортировка
От jian he
Тема [MASSMAIL]call ATPostAlterTypeParse inconsistency
Дата
Msg-id CACJufxGk8nBdamJFatbPawu9jqeqTz+wWgCG0YQMm+HW-dJcDg@mail.gmail.com
обсуждение исходный текст
Список pgsql-hackers
hi.
one minor issue.

within ATPostAlterTypeCleanup, we call ATPostAlterTypeParse:
ATPostAlterTypeParse(oldId, relid, InvalidOid,
(char *) lfirst(def_item),
wqueue, lockmode, tab->rewrite);


function ATPostAlterTypeParse is:
static void
ATPostAlterTypeParse(Oid oldId, Oid oldRelId, Oid refRelId, char *cmd,
List **wqueue, LOCKMODE lockmode, bool rewrite)

but tab->rewrite is an int.
so within ATPostAlterTypeCleanup we should call it like:

ATPostAlterTypeParse(oldId, relid, InvalidOid,
(char *) lfirst(def_item),
wqueue, lockmode, tab->rewrite != 0);

or

ATPostAlterTypeParse(oldId, relid, InvalidOid,
(char *) lfirst(def_item),
wqueue, lockmode, tab->rewrite > 0);



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

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Fix out-of-bounds in the function GetCommandTagName
Следующее
От: Michael Paquier
Дата:
Сообщение: Re: Stability of queryid in minor versions