Re: Code review in dsa.c

Поиск
Список
Период
Сортировка
От Zhang Mingli
Тема Re: Code review in dsa.c
Дата
Msg-id 959c08bb-3b3d-4143-b247-86527d5e5cc4@Spark
обсуждение исходный текст
Ответ на Code review in dsa.c  (Zhang Mingli <zmlpostgres@gmail.com>)
Ответы Re: Code review in dsa.c  (Nathan Bossart <nathandbossart@gmail.com>)
Список pgsql-hackers
HI,

On Jan 15, 2023, 23:43 +0800, Zhang Mingli <zmlpostgres@gmail.com>, wrote:
Hi, hackers

Found  some functions in dsa.c are not used anymore.

dsa_create
dsa_attach
dsa_get_handle
dsa_trim
dsa_dump

We once used dsa_create to create DSA and  it ’s all replaced by dsa_create_in_place since commit 31ae1638ce.
dsa_attach and dsa_get_handle cooperate with dsa_create.
dsa_trim and dsa_dump are introduced by DSA original commit 13df76a537 , but not used since then.

So, they are all dead codes, provide a patch to remove them.

Patch updated.
Forget to remove dsa_unpin in dsa.h, dsa_unpin is also not used since commit 13df76a537.
The gemel function dsa_pin is only used in pg_stat. Seems reasonable that we don’t need to call dsa_unpin in pg_stat.

Regards,
Zhang Mingli
Вложения

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

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Extracting cross-version-upgrade knowledge from buildfarm client
Следующее
От: Tom Lane
Дата:
Сообщение: Re: logrep stuck with 'ERROR: int2vector has too many elements'