[eclipselink-dev] bugs 327225, 326962, 327108

["Goerler, Adrian" <adrian.goerler@xxxxxxx>]

Hi,

https://bugs.eclipse.org/bugs/show_bug.cgi?id=327225 - MaxDB: JPQL date/time functions current_date, current_time are not mapped properly

reviewed by Tom, checked in at 8342, 8343. 

https://bugs.eclipse.org/bugs/show_bug.cgi?id=326962 - MaxDB: can't compare two parameters

reviewed by Tom, checked in at 8344.

https://bugs.eclipse.org/bugs/show_bug.cgi?id=327108 - make (portable) variant of test complexCountOnJoinedVariableOverManyToManySelfRefRelationship that uses order column in select list; skip original test on MaxDB

reviewed by Tom, checked in at 8345.

-Adrian

> -----Ursprüngliche Nachricht-----
> Von: eclipselink-dev-bounces@xxxxxxxxxxx 
> [mailto:eclipselink-dev-bounces@xxxxxxxxxxx] Im Auftrag von Tom Ware
> Gesendet: Donnerstag, 7. Oktober 2010 19:06
> An: Dev mailing list for Eclipse Persistence Services
> Betreff: Re: [eclipselink-dev] bug 327225 - MaxDB: JPQL 
> date/time functions current_date, current_time are not mapped properly
> 
> I am ok with this change.
> 
> -Tom
> 
> Goerler, Adrian wrote:
> > Hi all,
> >  
> > _https://bugs.eclipse.org/bugs/show_bug.cgi?id=327225_
> >  
> > besides implementing the current_date/time/timestamp 
> functions properly 
> > in the MaxDB platform, I am proposing to adjust the following test
> >  
> > --- 
> > 
> foundation/eclipselink.core.test/src/org/eclipse/persistence/t
> esting/tests/expressions/ExpressionUnitTestSuite.java 
> > (revision 8333)
> > +++ 
> > 
> foundation/eclipselink.core.test/src/org/eclipse/persistence/t
> esting/tests/expressions/ExpressionUnitTestSuite.java 
> > (working copy)
> > @@ -191,7 +191,7 @@
> >  
> >      protected void _addCurrentDateTest() {
> >          ExpressionBuilder builder = new ExpressionBuilder();
> > -        Expression expression = 
> > 
> builder.get("period").get("startDate").lessThan(builder.curren
> tDate());
> > +        Expression expression = 
> > 
> builder.get("period").get("startDate").lessThan(builder.curren
> tDateDate());
> >  
> > Employee.period.startDate has the type java.sql.Date and hence the 
> > currentDateDate() method (corresponding to current_date in 
> JPQL) should 
> > be used, I think. And MaxDB thinks so as well ;-).
> >  
> > OK if I proceed with this change?
> >  
> > -Adrian
> >  
> >  
> >  
> > SAP AG
> > Dietmar-Hopp-Allee 16, 69190 Walldorf, Germany
> >  
> > Sitz der Gesellschaft/Registered Office: Walldorf, Germany
> > Vorstand/SAP Executive Board: Henning Kagermann (Sprecher/CEO), Léo 
> > Apotheker (stellvertretender Sprecher/Deputy CEO), Werner 
> Brandt, Claus 
> > Heinrich, Gerhard Oswald, John Schwarz, Peter Zencke
> > Vorsitzender des Aufsichtsrats/Chairperson of the SAP 
> Supervisory Board: 
> > Hasso Plattner
> > Registergericht/Commercial Register Mannheim No HRB 350269
> >  
> >  
> > 
> > 
> > 
> --------------------------------------------------------------
> ----------
> > 
> > _______________________________________________
> > 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
>