From: | Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp> |
---|---|
To: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com> |
Cc: | Jeff Janes <jeff(dot)janes(at)gmail(dot)com>, Sawada Masahiko <sawada(dot)mshk(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Jesper Krogh <jesper(at)krogh(dot)cc>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: PENDING_LIST_CLEANUP_SIZE - maximum size of GIN pending list Re: HEAD seems to generate larger WAL regarding GIN index |
Date: | 2014-09-10 08:35:11 |
Message-ID: | 54100D3F.10102@lab.ntt.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
(2014/09/10 12:31), Fujii Masao wrote:
> On Wed, Sep 10, 2014 at 12:15 PM, Etsuro Fujita
> <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp> wrote:
>> (2014/09/09 22:17), Fujii Masao wrote:
>>> Attached is the updated version of the patch.
>> I took a quick review on the patch. It looks good to me,
>> but one thing I'm
>> concerned about is
>>
>> You wrote:
>>>>>> The attached patch introduces the GIN index storage parameter
>>>>>> "PENDING_LIST_CLEANUP_SIZE" which specifies the maximum size of
>>>>>> GIN pending list. If it's not set, work_mem is used as that maximum
>>>>>> size,
>>>>>> instead. So this patch doesn't break the existing application which
>>>>>> currently uses work_mem as the threshold of cleanup operation of
>>>>>> the pending list. It has only not to set PENDING_LIST_CLEANUP_SIZE.
>>
>> As you mentioned, I think it's important to consider for the existing
>> applications, but I'm wondering if it would be a bit confusing users to have
>> two parameters,
>
> Yep.
>
>> PENDING_LIST_CLEANUP_SIZE and work_mem, for this setting.
>> Wouldn't it be easy-to-use to have only one parameter,
>> PENDING_LIST_CLEANUP_SIZE? How about setting PENDING_LIST_CLEANUP_SIZE to
>> work_mem as the default value when running the CREATE INDEX command?
>
> That's an idea. But there might be some users who want to change
> the cleanup size per session like they can do by setting work_mem,
> and your idea would prevent them from doing that...
Why not use ALTER INDEX ... SET (PENDING_LIST_CLEANUP_SIZE= ...)? Maybe
I'm missing something, though.
> So what about introduing pending_list_cleanup_size also as GUC?
> That is, users can set the threshold by using either the reloption or
> GUC.
Yeah, that's an idea. So, I'd like to hear the opinions of others.
Thanks,
Best regards,
Etsuro Fujita
From | Date | Subject | |
---|---|---|---|
Next Message | Fabien COELHO | 2014-09-10 08:48:07 | Re: add modulo (%) operator to pgbench |
Previous Message | Mark Kirkwood | 2014-09-10 08:09:10 | Re: Scaling shared buffer eviction |