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
Список 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 по дате отправления:

Сайт использует файлы cookie для корректной работы и повышения удобства. Нажимая кнопку «Принять» или продолжая пользоваться сайтом, вы соглашаетесь на их использование в соответствии с Политикой в отношении обработки cookie ООО «ППГ», в том числе на передачу данных из файлов cookie сторонним статистическим и рекламным службам. Вы можете управлять настройками cookie через параметры вашего браузера