You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by "angela (JIRA)" <ji...@apache.org> on 2009/05/28 12:50:45 UTC
[jira] Resolved: (JCR-1418) Redesign NodeInfo.getReferences()
[ https://issues.apache.org/jira/browse/JCR-1418?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
angela resolved JCR-1418.
-------------------------
Resolution: Fixed
Fix Version/s: 2.0.0
fixed together with JCR 2.0 reference features that includes:
- only obtain references that match a specified name
- obtain weak references
added method to RepositoryService as suggested by marcel, but extended params.
due to the additional functionality and since jcr2spi now only and always calls the repositoryservice method, i decided to deprecate NodeInfo.getReferences instead of changing its semantics.... or should we simply remove it for 2.0?
> Redesign NodeInfo.getReferences()
> ---------------------------------
>
> Key: JCR-1418
> URL: https://issues.apache.org/jira/browse/JCR-1418
> Project: Jackrabbit Content Repository
> Issue Type: Improvement
> Components: jackrabbit-spi
> Affects Versions: 1.4
> Reporter: Marcel Reutegger
> Assignee: angela
> Priority: Minor
> Fix For: 2.0.0
>
>
> The method returns an array of PropertyIds. When there are lots of references this may become an problem. As with any other return value that potentially is large we should return an iterator.
> I suggest to redesign the handling of references in line with recent discussions how child infos are handled.
> - A NodeInfo implementation must either return the complete list of PropertyIds or null if it does not want to return the PropertyIds at that time.
> - Introduce a new method: Iterator<PropertyId> RepositoryService.getReferences(SessionInfo, NodeId)
> This has the following advantages:
> - loading of references can be delayed until it is really needed
> - large collections of references can be streamed through the SPI
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.