|
|
|
Created an attachment (id=597)
dbmapping.xml Created an attachment (id=598)
castor.properties Was doing a little debugging on this and figured out whats going on. For some
reason castor doesnt like the _employees field. It always give it an SQL Name of null. I added some debugging output right before the last line in the trace in SQLEngine and here's what I get for output: 100 [main] DEBUG org.exolab.castor.jdo.engine.SQLEngine - Stored procedure call The SQLname[] array is apparently null, while you can see the other ones arent. I tried the modified version of SQLEngine.java attached to bug 1288 - and it
seems this works. So I recommend committing this modification. To make things easier I will create and attach a patch to be applied to the current CVS code. I will also attach a updated version of my test case file DeptEmp.java, because Created an attachment (id=611)
Patch for org/exolab/castor/jdo/engine/SQLEngine.java Created an attachment (id=612)
corrected version of DeptEmp.java Martin, I will have a look at your patch tonight or tomorrow night. I'll keep
you posted about any progress I make through this report. Just added a dependency on bug 1628, as clearly this problem would be resolved
without any work if we committed bug 1628. I vaguely remember Mickael was dealing with this. However, upon looking at the
diff between the attached file and the current version in CVS, it will take some time to sort out because the time difference between the two version is almost 1.5 years. I simply don't have the time at the moment. Please look at my attached Patch file:
http://bugzilla.exolab.org/showattachment.cgi?attach_id=611 ![]() I already merged the changes of bug 1627 into the current CVS code. Martin, can you please provide me with the DDL you've use to craete the tables
DEPT and EMP. Actually, I am sure that my comment above about bug 628 should have gone
elsewhere .. ![]() ![]() Removed dependency on bug 1628 (as it was wrong).
The tables EMP and DEPT are part of the famous SCOTT example schema of oracle
databases. You can find a script to generate this schema at the OTN site: Gentlemen (iow dear fellow committers and wanna-bes), can I please get some
voting done on this patch ? I've integrated the patch, used Martin's donations as a test case, and run the Castor JDO test suite successfully. Any objections ? Any other comments ? The patch looks good, but I haven't applied it to be tested yet. As long as
it doesn't break any exiting tests, I think it's safe to commit. Patch supplied as provided (plus a minor addition to message.properties). Please
do not forget to close this bug once verified successfully. OK, thanks for the commit.
Could you please also correct my name in the CHANGELOG file - it's twice Will do en un segundo ... sorry for the name change. Though I'd advise to listen
to a song named' Family Name' by Prince ... ![]() Name spelling 'fix' committed. Sorry for the name change ...
![]()
|
DeptEmp.java