Re: Improve GetConfigOptionValues function

Поиск
Список
Период
Сортировка
От Nitin Jadhav
Тема Re: Improve GetConfigOptionValues function
Дата
Msg-id CAMm1aWZY38naak+RiG9XvOGEVa+NJ=Y0YYHnzEHdSSVDh0OfHw@mail.gmail.com
обсуждение исходный текст
Ответ на Improve GetConfigOptionValues function  (Nitin Jadhav <nitinjadhavpostgres@gmail.com>)
Ответы Re: Improve GetConfigOptionValues function  (Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>)
Список pgsql-hackers
Attaching the patch.

On Wed, Jan 18, 2023 at 1:21 PM Nitin Jadhav
<nitinjadhavpostgres@gmail.com> wrote:
>
> Hi,
>
> GetConfigOptionValues function extracts the config parameters for the
> given variable irrespective of whether it results in noshow or not.
> But the parent function show_all_settings ignores the values parameter
> if it results in noshow. It's unnecessary to fetch all the values
> during noshow. So a return statement in GetConfigOptionValues() when
> noshow is set to true is needed. Attached the patch for the same.
> Please share your thoughts.
>
> Thanks & Regards,
> Nitin Jadhav

Вложения

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

Предыдущее
От: Nitin Jadhav
Дата:
Сообщение: Improve GetConfigOptionValues function
Следующее
От: "Drouvot, Bertrand"
Дата:
Сообщение: Re: Helper functions for wait_for_catchup() in Cluster.pm