Postgres Pro
Downloads
Home   >   mailing lists

Re: pgsql: Try again to fix the way the scanjoin_target is used with partia - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: Try again to fix the way the scanjoin_target is used with partia
Date
Msg-id 9539.1466220369@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql: Try again to fix the way the scanjoin_target is used with partia  (Amit Kapila <amit.kapila16@gmail.com>)
Responses Re: pgsql: Try again to fix the way the scanjoin_target is used with partia  (Amit Kapila <amit.kapila16@gmail.com>)
List pgsql-committers
Tree view 
Amit Kapila <amit.kapila16@gmail.com> writes:
> Patch fixing the failures along with a new test case is attached with this
> mail

That can't possibly be right, because the PathTarget.sortgrouprefs field
is not a pointer to a Node.

I think the two lines in question are just a poorly-thought-through case
of premature optimization.  If removing them makes the failures go away
for me, that's what I plan to do.

            regards, tom lane

pgsql-committers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: pgsql: Try again to fix the way the scanjoin_target is used with partia
Next
From: Amit Kapila
Date:
Subject: Re: pgsql: Try again to fix the way the scanjoin_target is used with partia