Apache JDO Project - [jira] Commented: (JDO-402) checkRemoveGroup - removeGroup should

This is Interesting: Free IT Magazines  
Home > Archive > Apache JDO Project > August 2006 > [jira] Commented: (JDO-402) checkRemoveGroup - removeGroup should





You are viewing an archived Text-only version of the thread. To view this thread in it's original format and/or if you want to reply to this thread please [click here]

Author [jira] Commented: (JDO-402) checkRemoveGroup - removeGroup should
Craig Russell (JIRA)

2006-08-05, 7:11 pm

[ http://issues.apache.org/jira/brows...action_12426007 ]

Craig Russell commented on JDO-402:
-----------------------------------

Hi Andy,

The spec does say the returned Set is immutable but that doesn't necessarily mean that the underlying Set is immutable as well. That is, there is room for the Set to change based on the underlying groups. But I think it makes sense to return an immutable
Set that will never change, even if the underlying groups changes.


> 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/secur...nistrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira



Sponsored Links






Free braindumps | Software forum | Database administration forum

Copyright 2003 - 2008 webservertalk.com