From: Roy Lyseng
Date: January 24 2011 3:45pm
Subject: Re: bzr commit into mysql-trunk branch (roy.lyseng:3323) Bug#57623
List-Archive: http://lists.mysql.com/commits/129457
Message-Id: <4D3D9EB4.2050206@oracle.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 8bit

Hi Øystein,

thank you for the review.

On 21.01.11 10.33, Øystein Grøvlen wrote:
> Hi Roy,
>
> Thanks for the patch. I have verified that it fixes the issue at hand. My only
> question is that when prep_on_expr is later set, will not there be a need to do
> replace_subcondition on it?

Inside simplify_joins(), prep_on_expr is set based on on_expr, which by that 
time has already been transformed using replace_subcondition. prep_on_expr is 
formed by calling copy_andor_structure() on on_expr, which preserves "leaf" 
predicates of the original condition and adds new AND and OR nodes.

So I think there is no need for another replace_subcondition() call.

Thanks,
Roy