You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jdo-dev@db.apache.org by "Craig Russell (JIRA)" <ji...@apache.org> on 2006/03/29 18:47:18 UTC
[jira] Commented: (JDO-355) Remove extra tables/classes in the sql
and orm for mappings 5,6, and 7.
[ http://issues.apache.org/jira/browse/JDO-355?page=comments#action_12372279 ]
Craig Russell commented on JDO-355:
-----------------------------------
When the tests were written, I think we didn't know exactly what we were going to test, and made extra classes persistent in order to have some flexibility.
Now that the tests are complete, it's not clear to me that we will ever extend the test cases so we don't need to keep the extra classes in the sql and orm.
So it's ok with me to remove the extras.
> Remove extra tables/classes in the sql and orm for mappings 5,6, and 7.
> -----------------------------------------------------------------------
>
> Key: JDO-355
> URL: http://issues.apache.org/jira/browse/JDO-355
> Project: JDO
> Type: Improvement
> Components: tck20
> Versions: JDO 2 rc1
> Reporter: Michael Watzek
> Assignee: Michael Watzek
> Priority: Minor
> Fix For: JDO 2 final
>
> I noticed that there is a little mismatch between test case MakePersistent, the .sql files, and the .orm files for mappings 5, 6, 7 wrt. which persistence capable classes are needed:
> - Test case MakePersistent uses pc class PCPoint only.
> - The .sql files create tables PCPoint and PCRect.
> - The .orm files specifiy metadata for pc classes PCPoint, PCPoint2, PCRect, and PrimitiveTypes.
> This holds true for application identity and datastore identity as well.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira