[ 
https://issues.apache.org/jira/browse/JCR-1865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12649292#action_12649292
 ] 

Felix Meschberger commented on JCR-1865:
----------------------------------------

> I'm not sure if this would be fully backward compatible

Well, you already renamed the interface, right ? So you already have kind of a 
backwards compatibility issue.

In addition, there has not been an official API yet (interfaces "hidden" in 
implementation code should not be called API ;-) ), so IMHO there is no need to 
worry.

> * single method "gc()": multiple repositories can use the same data store 
> (sharing the data store). In that case,
> you need to call mark() of all repositories before calling sweep() (on the 
> first repository).

As Jukka already said, sharing the store among multiple repositories is an 
implementation detail of the data store and is of no concern to the 
administrator (+1 for making it a management API BTW). Synchronization among 
the repositories sharing the data store is the task of the data store.

> our lower level APIs

What is "lower level APIs" ?

> Add the Data Store to the Jackrabbit API
> ----------------------------------------
>
>                 Key: JCR-1865
>                 URL: https://issues.apache.org/jira/browse/JCR-1865
>             Project: Jackrabbit
>          Issue Type: Improvement
>          Components: jackrabbit-core
>            Reporter: Thomas Mueller
>            Assignee: Thomas Mueller
>            Priority: Minor
>         Attachments: api.patch, core.patch
>
>
> Currently, the garbage collection is not part of the Jackrabbit API. However, 
> the data store garbage collection must be used once in a while if the data 
> store is enabled. I propose to add the required interfaces to the Jackrabbit 
> API. This will also allow to call garbage collection using RMI.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to