|
From: Marc de H. <m.d...@to...> - 2005-08-24 09:48:18
|
Thank you Fred for looking to our code and implementing it in the newest version of hsqldb. Nevertheless, we've encountered one little problem. When using a left join, hsqldb gives an exception when NOT is used in the join condition. In the last version of the files we sent, we fixed this by changing Expression.java at line 3806 by adding || exprType == Expression.NOT. If you don't have a particular reason to not use this code, could you please add this part to Expression.java (see also attached file). We think everything else works fine. Thanks again, Huug Ouwersloot Marc de Hoop - TOPdesk >Thanks Marc and Huug, > >I have incorporated your changes for DOW and RIGHT OUTER as submitted in the >patch. > >For SET NOT NULL I wrote a new implementation that checks the existing rows >and primary key. > >For Aggregates and CASE WHEN, I traced the problem to the original code and >fixed it (in Expression.updateAggregatingValue()). > >Have just committed the changes to the CVS hsqldb-dev module > >Fred > >----- Original Message ----- >From: "fredt" <fr...@us...> >To: <hsq...@li...> >Sent: 16 August 2005 16:04 >Subject: Re: [Hsqldb-developers] hsqldb - added features like Right Join >support, extract(dayofweek) and another bug-fix to the aggregates/build-in >types problem > > >Thanks for the effort. I have done the first review of the patches: > >DOW: This looks fine. >SET NOT NULL: This has yet to check for values in existing rows. We have a >similar test for another alter command and can reuse it. >RIGHT OUTER: This looks fine. I need to check and test it further. >Aggregates and CASE WHEN: The change to Column.java is out of context, as >the Compare method is for comparison of column values of the same time. >Needs more work. > >I will review the changes further and add some tests in the next couple of >days. > >Fred > > > >----- Original Message ----- >From: "Marc de Hoop" <m.d...@to...> >To: <hsq...@li...> >Sent: 15 August 2005 15:14 >Subject: [Hsqldb-developers] hsqldb - added features like Right Join >support, extract(dayofweek) and another bug-fix to the aggregates/build-in >types problem > > >Hello list, > >We've made some other updates to hsqldb. We'll be posting them to the >patch tracker in a minute. >We've developed Right Join support, extracting a day of week of a >datetime field ( extract(dayofweek, <fieldname>) which returns the >values: sunday = 1, monday = 2, etc.) and a better version of our >bug-fix to aggregates / build-in types. In the files we'll post, our >previous feature (set [not] null ) is also included since we don't have >got any comment yet. >We'd appreciate it if someone would review our code. > >Yours, > >Marc de Hoop >Huug Ouwersloot >- TOPdesk > > >------------------------------------------------------- >SF.Net email is Sponsored by the Better Software Conference & EXPO >September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices >Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA >Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf >_______________________________________________ >hsqldb-developers mailing list >hsq...@li... >https://lists.sourceforge.net/lists/listinfo/hsqldb-developers > > > >------------------------------------------------------- >SF.Net email is Sponsored by the Better Software Conference & EXPO >September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices >Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA >Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf >_______________________________________________ >hsqldb-developers mailing list >hsq...@li... >https://lists.sourceforge.net/lists/listinfo/hsqldb-developers > > > >------------------------------------------------------- >SF.Net email is Sponsored by the Better Software Conference & EXPO >September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices >Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA >Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf >_______________________________________________ >hsqldb-developers mailing list >hsq...@li... >https://lists.sourceforge.net/lists/listinfo/hsqldb-developers > > > |