[ http://issues.apache.org/jira/browse/JDO-402?page=comments#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/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira
