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 "Ilan Kirsh (JIRA)" <ji...@apache.org> on 2006/07/30 04:18:14 UTC
[jira] Commented: (JDO-402) checkRemoveGroup - removeGroup should
be before getGroups
[ http://issues.apache.org/jira/browse/JDO-402?page=comments#action_12424341 ]
Ilan Kirsh commented on JDO-402:
--------------------------------
Thanks for the fix. I assume that the old code should work fine if the implementation returns from getGroup a shared set that is still used by the FetchPlan.
> checkRemoveGroup - removeGroup should be before getGroups
> ---------------------------------------------------------
>
> Key: JDO-402
> URL: http://issues.apache.org/jira/browse/JDO-402
> Project: JDO
> Issue Type: Bug
> Components: tck20
> Affects Versions: JDO 2 final
> Reporter: Ilan Kirsh
> Assigned To: Craig Russell
> Fix For: JDO 2 maintenance release 1
>
> Attachments: JDO-402.patch
>
>
> In org.apache.jdo.tck.api.persistencemanager.fetchplan.FetchPlanInterface, lines 281-282,
> it probably has to be:
> fp.removeGroup("default");
> Collection groups = fp.getGroups();
> Instead of:
> Collection groups = fp.getGroups();
> fp.removeGroup("default");
--
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