Re: [eclipselink-dev] Bug 305713-OuterJoinExpressionHolderlinearization

["Andrei Ilitchev" <andrei.ilitchev@xxxxxxxxxx>]

Checked the updated patch into trunk (2.1)

Updated SQLSelectStatement to accommodate James' changes (outer join for map
key).
Had to alter a couple of test classes:
JoinedAttributeTestHelper - to take into account that for OneToOne mapping 
in
case the foreign key is null the value holder in the read object is always
triggered;
ReportQueryMultipleReturnTestSuite.testReturnRootObject - had to change the
reference class, see the comment in the test.

----- Original Message ----- 
From: "Andrei Ilitchev" <andrei.ilitchev@xxxxxxxxxx>
To: "Dev mailing list for Eclipse Persistence Services" 
<eclipselink-dev@xxxxxxxxxxx>
Sent: Tuesday, April 13, 2010 6:40 PM
Subject: Re: [eclipselink-dev] Bug 
305713-OuterJoinExpressionHolderlinearization order can beincorrectduring 
ansi joins


> Rolled back due to code collision in SQLSelectStatement class.
>
> ----- Original Message ----- 
> From: "Andrei Ilitchev" <andrei.ilitchev@xxxxxxxxxx>
> To: "Dev mailing list for Eclipse Persistence Services" 
> <eclipselink-dev@xxxxxxxxxxx>
> Sent: Tuesday, April 13, 2010 4:06 PM
> Subject: Re: [eclipselink-dev] Bug 
> 305713 -OuterJoinExpressionHolderlinearization order can be 
> incorrectduring ansi joins
>
>
> > Checked the patch into trunk.
> > Reviewed by James.
> > ----- Original Message ----- 
> > From: "Andrei Ilitchev" <andrei.ilitchev@xxxxxxxxxx>
> > To: "Dev mailing list for Eclipse Persistence Services" 
> > <eclipselink-dev@xxxxxxxxxxx>
> > Sent: Tuesday, April 13, 2010 11:36 AM
> > Subject: [eclipselink-dev] Bug 305713 - 
> > OuterJoinExpressionHolderlinearization order can be incorrect during ansi 
> > joins
> >
> >
> > > https://bugs.eclipse.org/bugs/show_bug.cgi?id=305713
> > > Reopened this bug, posted the patch.
> > >
> > > Please send your comments.
> > >
> > > Thanks,
> > > Andrei
> > > _______________________________________________
> > > eclipselink-dev mailing list
> > > eclipselink-dev@xxxxxxxxxxx
> > > https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
> >
> > _______________________________________________
> > eclipselink-dev mailing list
> > eclipselink-dev@xxxxxxxxxxx
> > https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>
> _______________________________________________
> eclipselink-dev mailing list
> eclipselink-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev