The DN4 api has changed, but I think that part is worked out. When I merged your changes, I mistakenly registered the metadataListener after the call to createSchema() had already taken place. Made some minor tweaks and it works like a charm. Thanks Dan!
I'll finish cleaning the branch up and submit a PR soon. Jeremy D. Branham Tel: **DOTNET -----Original Message----- From: Dan Haywood [mailto:[email protected]] Sent: Thursday, March 19, 2015 3:51 AM To: users Subject: Re: Apache Isis 1.9.0 + DN 4.0.4 The syntax that works everywhere is without quotes for the schema, but with quotes for the table... CREATE TABLE simple."SimpleObject" ... I did complete a ticket ISIS-1064 [1] that will automatically create the schema; core of the implementation is [2] and [3]. So you might want to check that your patch has merged in that stuff (and that the DN API hasn't changed in 4.0.x). Cheers Dan [1] https://issues.apache.org/jira/browse/ISIS-1064 [2] https://github.com/apache/isis/blob/55e569c9bbadddd3de1b5e6e703181400424658d/core/runtime/src/main/java/org/apache/isis/objectstore/jdo/datanucleus/DataNucleusApplicationComponents.java#L131 [3] https://github.com/apache/isis/blob/master/core/runtime/src/main/java/org/apache/isis/objectstore/jdo/datanucleus/CreateSchemaObjectFromClassMetadata.java On 19 March 2015 at 08:38, Branham, Jeremy [HR] <[email protected] > wrote: > Thanks Dan - > I did make many changes from the earlier branch. I'm working through a > few things to make it cleaner, and will review the Apache committing > guidelines to save some review time. > > I did notice an issue with HSQL not accepting the schema when creating > new tables. I've tried it with and without quotes, but the only thing > that has worked so far is removing the schema altogether. > > > java.sql.SQLException: > invalid schema name: simple in statement [CREATE TABLE > "simple"."SimpleObject" ......... > > > > > Jeremy D. Branham > Tel: **DOTNET > > -----Original Message----- > From: Dan Haywood [mailto:[email protected]] > Sent: Thursday, March 19, 2015 3:19 AM > To: users > Subject: Re: Apache Isis 1.9.0 + DN 4.0.4 > > Hi Jeremy, > > Thanks for doing this... yes, if you create a PR, then I'll review and > merge. (I do have a branch from your earlier work, but if you've > tidied it up then so much the better). > > Cheers > Dan > > > On 19 March 2015 at 00:56, Branham, Jeremy [HR] < > [email protected] > > wrote: > > > I have merged the latest Apache Isis master with my fork [1] > > including DataNucleus 4.0.4 > > > > There are some adjustments I can revert to make a pull request > > easier, but should be ready to request it in the next couple of days > > if that is > ok. > > There are a lot of changes, but all tests are currently passing. > > > > > > [1] https://github.com/jdbranham/isis > > > > > > Jeremy D. Branham > > Technology Architect > > Sprint University Performance Support > > > > > > ________________________________ > > > > This e-mail may contain Sprint proprietary information intended for > > the sole use of the recipient(s). Any use by others is prohibited. > > If you are not the intended recipient, please contact the sender and > > delete all copies of the message. > > > > ________________________________ > > This e-mail may contain Sprint proprietary information intended for > the sole use of the recipient(s). Any use by others is prohibited. If > you are not the intended recipient, please contact the sender and > delete all copies of the message. > ________________________________ This e-mail may contain Sprint proprietary information intended for the sole use of the recipient(s). Any use by others is prohibited. If you are not the intended recipient, please contact the sender and delete all copies of the message.
