Date: Thu, 19 Dec 2019 14:14:10 +0100
From: Jehan-Guillaume de Rorthais <jgdr@dalibo.com>
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: vignesh C <vignesh21@gmail.com>, Peter Eisentraut
 <peter.eisentraut@2ndquadrant.com>, PostgreSQL Hackers
 <pgsql-hackers@lists.postgresql.org>
Subject: Re: segmentation fault when cassert enabled
Message-ID: <20191219141410.79745d26@firost>
In-Reply-To: 
 <CAA4eK1+QF_-T6cnTcHHTKxLoR4BYyN-2xtj-SZTXL4ipdYYryg@mail.gmail.com>
References: <20191025175929.7e90dbf5@firost>
	<18714.1572020918@sss.pgh.pa.us>
	<20191105172918.3e32a446@firost>
	<c75077cd-a1a7-9a2d-5dc6-4d3d8acfaafa@2ndquadrant.com>
	<20191125155519.5338e3c9@firost>
	<CAA4eK1+9vQb34faKLJbY6KD62HZOJ5Jm9PzoZGiK_9J7cvyDdw@mail.gmail.com>
	<CALDaNm0kVT3+ku99M6=Wy9X=3=-H8jp_Yt7WVuTraPuREZugxQ@mail.gmail.com>
	<CAA4eK1LGNKWDBQ7ktxfQ6vL=f8t_05b_4pSch4Hr=CNj-S-a7Q@mail.gmail.com>
	<32a64f45-6c01-d2bf-d90a-49ff4bfedcfb@2ndquadrant.com>
	<20191216164639.4b29efd4@firost>
	<CAA4eK1LdJ0wdouSk3fHXJrsQuEjw6KSR9xxFWrzAFyLJV8VU1Q@mail.gmail.com>
	<CALDaNm3_e20vjB8EQL760zC_V_Y8+usjWgyHfpz4jYezOQTFOw@mail.gmail.com>
	<CAA4eK1+QF_-T6cnTcHHTKxLoR4BYyN-2xtj-SZTXL4ipdYYryg@mail.gmail.com>
Organization: Dalibo
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Wed, 18 Dec 2019 08:46:01 +0530
Amit Kapila <amit.kapila16@gmail.com> wrote:

> On Tue, Dec 17, 2019 at 6:01 PM vignesh C <vignesh21@gmail.com> wrote:
> >
> > On Tue, Dec 17, 2019 at 10:09 AM Amit Kapila <amit.kapila16@gmail.com>
> > wrote:  
> > >
> > > Attached patch with updated commit message based on suggestions.  I am
> > > planning to commit this tomorrow unless there are more comments.
> > >  
> >
> > While testing the patch on back versions, I found that the patch does
> > not apply on PG 11 & PG 10 branch. Attached patch has the changes for
> > PG 11 & PG 10 branch. Only difference in the patch was that table_open
> > needed to be changed to heap_open. I have verified the patch on back
> > branches and found it to be working. For PG 12 & current the previous
> > patch that Amit need to be used, I'm not reattaching here.
> >  
> 
> Pushed.

Thanks!