Re: Small patch: Change calling convention for ShmemInitHash (and fix possible bug)

Поиск
Список
Период
Сортировка
От Robert Haas
Тема Re: Small patch: Change calling convention for ShmemInitHash (and fix possible bug)
Дата
Msg-id CA+TgmoagyCi_Oqen3KisK-_W8JO-v2YLuocAOa4UVkwoeDr_Hg@mail.gmail.com
обсуждение исходный текст
Ответ на Small patch: Change calling convention for ShmemInitHash (and fix possible bug)  (Aleksander Alekseev <a.alekseev@postgrespro.ru>)
Ответы Re: Small patch: Change calling convention for ShmemInitHash (and fix possible bug)  (Aleksander Alekseev <a.alekseev@postgrespro.ru>)
Re: Small patch: Change calling convention for ShmemInitHash (and fix possible bug)  (Michael Paquier <michael.paquier@gmail.com>)
Список pgsql-hackers
On Thu, Mar 24, 2016 at 9:50 AM, Aleksander Alekseev
<a.alekseev@postgrespro.ru> wrote:
> I would like to continue discussion regarding changing calling
> convention for ShmemInitHash procedure:
>
> http://www.postgresql.org/message-id/CA+TgmoZm=Uowt8a_XaSfooGwufeeLJ861NTADiCEOpyFehV8Wg@mail.gmail.com
>
> Currently this procedure has two arguments --- init_size and max_size.
> But since shared hash tables have fixed size there is little sense to
> pass two arguments. In fact currently ShmemInitHash is always called
> with init_size == max_size with only one exception, InitLocks procedure
> (see lock.c), which I believe is actually a bug.

No, I think we left it that way on purpose.  I don't remember the
discussion exactly, but I don't think it's hurting anything.

> Patch is attached.
>
> What do you think?

I don't think this actually buys us anything.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



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

Предыдущее
От: Robert Haas
Дата:
Сообщение: Re: Small patch: fix code duplication in heapam.c
Следующее
От: Robert Haas
Дата:
Сообщение: Re: Support for N synchronous standby servers - take 2