[ 
https://issues.apache.org/jira/browse/SLING-6263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15651105#comment-15651105
 ] 

Bertrand Delacretaz commented on SLING-6263:
--------------------------------------------

I disagree with changing the javadocs or renaming, this is changing the purpose 
of the {{SlingRepositoryInitializer}}. 

Executing such services as they appear is probably useful as an additional 
option but the main role of the {{SlingRepositoryInitializer}} is to be 
executed _before_ making the {{SlingRepository}} service available - as 
discussed at 
https://lists.apache.org/thread.html/a55778fabe3ff5a5403eb5f852e10a2b4184b05e01366959d88c821c@%3Cdev.sling.apache.org%3E
 - so we need a reliable way for that to happen for the 
{{SlingRepositoryInitializer}} services that require that.

I don't have time to implement that now, so not opposed to a release that 
doesn't have that feature, provided there's a backwards compatible migration 
path once we implement that.

> Dependency problem between RepositoryInitializer and 
> AbstractSlingRepositoryManager
> -----------------------------------------------------------------------------------
>
>                 Key: SLING-6263
>                 URL: https://issues.apache.org/jira/browse/SLING-6263
>             Project: Sling
>          Issue Type: Bug
>          Components: JCR
>            Reporter: Carsten Ziegeler
>            Assignee: Carsten Ziegeler
>            Priority: Blocker
>             Fix For: JCR Base 2.4.2
>
>
> As we can currently see on our Jenkins:
> https://builds.apache.org/job/sling-launchpad-builder-1.8/lastSuccessfulBuild/artifact/target/_-32861/sling/logs/error.log
> there is a dependency problem with the repository and repoinit.
> When the repository is started, there is no RepositoryInitializer
> service and the repository starts up fine. When the
> RepositoryInitializer arrives later it is completely ignored.
> I'm not 100% sure how to fix this, easiest fix would be to make a
> RepositoryInitializer required, but then this would mean only one of
> them is required.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to