Richard Guo writes:
> However, for 'above_sjinfo' case, we should not use
> othersj->syn_righthand, because othersj->syn_righthand contains relids
> in sjinfo's righthand which should not be nulled by othersj after the
> commutation. It seems what we should use here is sjinfo->syn_lefthand.
I h
On Fri, Feb 10, 2023 at 11:08 AM Richard Guo wrote:
> On Thu, Feb 9, 2023 at 11:55 PM Tom Lane wrote:
>
>> Richard Guo writes:
>> > This query would trigger the Assert() in search_indexed_tlist_for_var.
>> > So I wonder that we should use othersj->syn_righthand here.
>>
>> There are two such ca
On Thu, Feb 9, 2023 at 11:55 PM Tom Lane wrote:
> Richard Guo writes:
> > This query would trigger the Assert() in search_indexed_tlist_for_var.
> > So I wonder that we should use othersj->syn_righthand here.
>
> There are two such calls in deconstruct_distribute_oj_quals ...
> don't they both n
I wrote:
> Richard Guo writes:
>> It seems to me there is oversight here. Actually in next level up this
>> othersj would null all the relids in its syn_righthand, not only the
>> relids in its min_righthand.
> Good point. I think this code originated before it was clear to me
> that nullingrel
Richard Guo writes:
> It seems to me there is oversight here. Actually in next level up this
> othersj would null all the relids in its syn_righthand, not only the
> relids in its min_righthand.
Good point. I think this code originated before it was clear to me
that nullingrels would need to fo
