[EMAIL PROTECTED] wrote: > Daniel John Debrunner (JIRA) writes: > > Thanks Richard, I'll review the patch, but as a quick note Derby > > already supports SQRTand ABS (also added in your patch). > > I think all the others are new to Derby. > > Yes, these are implemented in a different section to the others > which is why I didn't realise until after I'd submitted the patch. > More importantly, these are the only ones which actually seem to > work properly, as can be seen from the following transcript: > > ij version 10.3 > ij> connect 'jdbc:derby:test;create=true'; > ij> create table simple_tab ( > x integer > ); > > 0 rows inserted/updated/deleted > ij> insert into simple_tab values (1); > 1 row inserted/updated/deleted > ij> select acos(x) y from simple_tab; > Y > ---------------------- > 0.0 > > 1 row selected > ij> create view simple_acos_view as ( > select acos(x) y from simple_tab > ); > > ERROR 42X94: OBJECT '2fb0c07e-010d-6f90-2909-00000012ff18' does not exist. > ij> > quit; > > The attached sql files show that this works across all functions, > against the current svn head. It's not the changes I've made, because > I've just rebuilt from the originals. If you can point me in the right > direction, I'll take a look and try to submit a patch. Otherwise, I'll > file the appropriate bug report. The current stable version appears to > be unaffected,
Thas looks like a bug in the new builtin function setup, not sure where to look, my guess is some code is assuming that these descriptors are perisistent when they are just in-memory descriptors. Can you add a bug for this. Richard, FYI, you may want to keep an eye on DERBY-1802, Susan has contributed a JUnit test for these new builting functions, it needs some cleanup before I can commit it, but once it's in the codeline, you may consider adding additional cases for the functions you have added. Thanks, Dan.
