[
https://issues.apache.org/jira/browse/JCR-1513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12589591#action_12589591
]
Julian Reschke commented on JCR-1513:
-------------------------------------
Proposal:
1) Allow checkin to return a NodeId *or* null (for easier migration of existing
SPI impls)
2) Use the non-null return value in JCR2SPI to avoid fetching the base version.
Then finally:
3) Get rid of the support for null return values.
> RepositoryService.checkin should return information about newly created
> version
> -------------------------------------------------------------------------------
>
> Key: JCR-1513
> URL: https://issues.apache.org/jira/browse/JCR-1513
> Project: Jackrabbit
> Issue Type: Improvement
> Components: jackrabbit-spi
> Reporter: Julian Reschke
> Assignee: Julian Reschke
>
> We have a mismatch between Node.checkin(), which returns a new Version
> object, and RepositoryService.checkin(), which returns void.
> Client of SPI, such as JCR2SPI, thus will have to make an additional request
> for the base version property, with the obvious drawbacks (another call, and
> a potential cause for a race condition).
> Proposal: change the return code to NodeId.
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
