Re: ObjectIdGetDatum() missing from SearchSysCache*() callers

Поиск
Список
Период
Сортировка
От Zhang Mingli
Тема Re: ObjectIdGetDatum() missing from SearchSysCache*() callers
Дата
Msg-id 0a6ee217-6958-4f13-9af8-61904f677f09@Spark
обсуждение исходный текст
Ответ на ObjectIdGetDatum() missing from SearchSysCache*() callers  (Michael Paquier <michael@paquier.xyz>)
Ответы Re: ObjectIdGetDatum() missing from SearchSysCache*() callers  (Zhang Mingli <zmlpostgres@gmail.com>)
Список pgsql-hackers
Дерево обсуждения
Hi

Regards,
Zhang Mingli
On Jul 17, 2023 at 19:10 +0800, Michael Paquier <michael@paquier.xyz>, wrote:
Hi all,

While scanning the code, I have noticed that a couple of code paths
that do syscache lookups are passing down directly Oids rather than
Datums. I think that we'd better be consistent here, even if there is
no actual bug.

I have noticed 11 callers of SearchSysCache*() that pass down
an Oid instead of a Datum.

Thoughts or comments?
--
Michael
LGTM, and there are two functions missed, in sequence_options

 pgstuple = SearchSysCache1(SEQRELID, relid);

Shall we fix that too?

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

Предыдущее
От: Jelte Fennema
Дата:
Сообщение: Re: [EXTERNAL] Re: Add non-blocking version of PQcancel
Следующее
От: Zhang Mingli
Дата:
Сообщение: Re: ObjectIdGetDatum() missing from SearchSysCache*() callers