From: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Aleksander Alekseev <a(dot)alekseev(at)postgrespro(dot)ru>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Small patch: Change calling convention for ShmemInitHash (and fix possible bug) |
Date: | 2016-03-25 12:37:01 |
Message-ID: | CAB7nPqRcV6RrN_VuV2zRV0qhkamxHTUAJFJmfnJnFTL+w6eBpg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Mar 25, 2016 at 9:17 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Thu, Mar 24, 2016 at 9:50 AM, Aleksander Alekseev
> <a(dot)alekseev(at)postgrespro(dot)ru> wrote:
>> 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.
My instinct is telling me that this is useful in this shape for
plugins, and that it has been designed on purpose for that.
--
Michael
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2016-03-25 12:41:51 | Re: Breakage with VACUUM ANALYSE + partitions |
Previous Message | Aleksander Alekseev | 2016-03-25 12:33:08 | Re: Small patch: Change calling convention for ShmemInitHash (and fix possible bug) |