From: "R, Siva" <sivasubr@amazon.com>
To: Alexander Korotkov <a.korotkov@postgrespro.ru>
CC: pgsql-hackers <pgsql-hackers@postgresql.org>
Subject: Re: Bug in gin insert redo code path during re-compression of empty
 gin data leaf pages
Thread-Topic: Bug in gin insert redo code path during re-compression of empty
 gin data leaf pages
Thread-Index: AQHUHhriTtCi9KoAzkiIHCifB95wm6SUru8AgAHAO4CAABK9gIAAJso2
Date: Thu, 19 Jul 2018 15:17:09 +0000
Message-ID: <1532013429633.24570@amazon.com>
References: <1531867212836.63354@amazon.com>
 <CAPpHfdu9p8fpNsBCcdYxhBpXhe2_g3CDKjTKhh5BCEDuz6JdiA@mail.gmail.com>
 <CAPpHfds627C9L-qSOBMqg3-L7Diha5vuaOFSRNE2J5XF3wwsDA@mail.gmail.com>,<CAPpHfduzgM_YtgFMDRFwEtD2GbR9McPfGDRA1+Zcc6NvMQxxxw@mail.gmail.com>
In-Reply-To: 
 <CAPpHfduzgM_YtgFMDRFwEtD2GbR9McPfGDRA1+Zcc6NvMQxxxw@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Precedence: Bulk
Precedence: bulk

Hi Alexander!=0A=
=0A=
> On Thu, Jul 19, 2018 at 2:43 PM Alexander Korotkov <a.korotkov@postgrespr=
o.ru> wrote:=0A=
> It appears that we need to handle empty posting list pages also  in=A0dis=
assembleLeaf().=A0 Updated patch is attached.=A0 I'm =0A=
> going to commit  it.=0A=
Thank you so much for reviewing the patch! You are right, we need to handle=
 the regular insert/vacuum path for the empty page as well, thanks for fixi=
ng that. The updated patch looks good to me.=0A=
=0A=
>BTW, what is your full name for the commit message? =0A=
My full name is "Sivasubramanian Ramasubramanian". =0A=
Email : sivasubr@amazon.com=0A=
=0A=
Thanks again!=0A=
=0A=
Best=0A=
Siva=0A=
=0A=
=0A=
=0A=
=0A=
=A0      =