Re: Removing Inner Joins - Mailing list pgsql-hackers

From Atri Sharma
Subject Re: Removing Inner Joins
Date
Msg-id B90250E0-1231-4810-885C-EB0E117F649F@gmail.com
Whole thread Raw
In response to Re: Removing Inner Joins  (Hannu Krosing <hannu@2ndQuadrant.com>)
Responses Re: Removing Inner Joins
List pgsql-hackers

Sent from my iPad

On 10-Jul-2013, at 13:11, Hannu Krosing <hannu@2ndQuadrant.com> wrote:

> On 07/10/2013 09:18 AM, Atri Sharma wrote:
>>> Can you please post an example of such a join removal? I mean a query before
>>> and after the removal. Thanks,
>> Courtesy Robert Haas:
>>
>> SELECT foo.x, foo.y, foo.z FROM foo WHERE foo.x = bar.x
>>
>> Conditions:
>>
>> 1) foo.x is not null.
> I guess that this is also not needed. you can just remove rows where
>
> foo.x is null
>
> That is, replace the join with "foo.x is not null"
>>
>> 2) foo (x) is a foreign key referencing bar (x).
>>
>> We can ignore bar completely in this case i.e. avoid scanning bar.
>>
>> Regards,
>>
>> Atri
>>
>>
>> --
>> Regards,
>>
>> Atri
>> l'apprenant
>>
>>
>
>
>

I discussed with RhodiumToad and was exploring potential design methods with which we can solve the above problem. My
focusis to add support for foreign key detection in planner and allow planner to make decisions based on it. 

It wouldn't be too much of a cost to maintain the foreign key column and the referenced table. The main issue, as
pointedout by RHodiumToad is primarily that, between the generation of the plan, which is made with accordance to the
foreignkey presence, and the execution of the plan, we may get into an inconsistent state when the corresponding row is
deletedor constraints are changed and fk trigger has not yet run and detected those changes. 

I was thinking of row level locks,which are done by the fk trigger.Is there any way we can modify the fk trigger to
forciblyrun? Or add an 'looked at' bit, which is reset when a table/row is changed, and set by the fk trigger when it
runson that table? 

I am just thinking wild here, please let me know your thoughts, feedback and ideas.

Regards,

Atri


pgsql-hackers by date:

Previous
From: Martijn van Oosterhout
Date:
Subject: Re: Patch to add support of "IF NOT EXISTS" to others "CREATE" statements
Next
From: Hannu Krosing
Date:
Subject: Re: Removing Inner Joins