|From:||Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>|
|To:||Michael Paquier <michael(dot)paquier(at)gmail(dot)com>|
|Cc:||Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>|
|Subject:||Re: Setting pd_lower in GIN metapage|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On 2017/09/11 18:13, Michael Paquier wrote:
> On Mon, Sep 11, 2017 at 5:40 PM, Amit Langote wrote:
>> On 2017/09/10 15:22, Michael Paquier wrote:
>>> Coordinating efforts here would be nice. If you, Amit K, are taking
>>> care of a patch for btree and hash, would you, Amit L, write the part
>>> for GIN, BRIN and SpGist? This needs a careful lookup as many code
>>> paths need a lookup so it may take time. Please note that I don't mind
>>> double-checking this part if you don't have enough room to do so.
>> Sorry, I didn't have time today to carefully go through the recent
>> discussion on this thread (starting with Tom's email wherein he said he
>> set the status of the patch to Waiting on Author). I will try tomorrow.
> Thanks for the update! Once you get to this point, please let me know
> if you would like to work on a more complete patch for brin, gin and
> spgist. If you don't have enough room, I am fine to produce something.
I updated the patches for GIN, BRIN, and SP-GiST to include the following
1. Pass REGBUF_STNADARD flag when registering the metapage buffer
2. Pass true for page_std argument of log_newpage() or
log_newpage_buffer() when using it to log the metapage
3. Update comment near where pd_lower is set in the respective metapages,
similar to what Amit K did in his patches for btree and hash metapages.
Did I miss something from the discussion?
|Next Message||Ashutosh Bapat||2017-09-12 10:56:38||Re: Partition-wise join for join between (declaratively) partitioned tables|
|Previous Message||Thomas Munro||2017-09-12 10:15:34||Re: Supporting huge pages on Windows|