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

Carsten Ziegeler commented on SLING-5337:
-----------------------------------------

With SLING-4892 I removed the usage of the startup handler from the settings 
bundle.
I think in general, we should avoid dependencies to the startup handler / 
Launchpad API. Maybe we should revisit all our code and check whether we can 
implement it without a dependency to the StartupHandler.

The startup handler is of use of Sling's launchpad as it provides a means to 
distinguish between install, update, and restart and allows update logic to run 
and other parts in the system to identify whether the startup is done.

> Extract Startup API from Launchpad API
> --------------------------------------
>
>                 Key: SLING-5337
>                 URL: https://issues.apache.org/jira/browse/SLING-5337
>             Project: Sling
>          Issue Type: Improvement
>          Components: Launchpad
>    Affects Versions: Launchpad API 1.2.0
>            Reporter: Oliver Lietz
>            Assignee: Oliver Lietz
>             Fix For: Launchpad API 1.3.0, Startup API 1.0.0
>
>
> {{StartupHandler}}, {{StartupListener}}, {{StartupMode}} and 
> {{StartupService}} in {{org.apache.sling.launchpad.api}} should be moved to 
> {{org.apache.sling.startup.api}} (keeping extending but deprecated classes 
> there for backward compatibility) to decouple Startup from Launchpad.
> A Startup implementation in Sling needs to be provided by every launcher 
> (Launchpad, Crankstart, Karaf)\[1\] whereas {{LaunchpadContentProvider}} is 
> only useful in contexts of Sling's own launcher Launchpad for installation 
> purpose.
> [~cziegeler], [~bdelacretaz]: anything from your POVs to consider?
> \[1\] We discussed to make Startup optional in {{Settings}} but that won't 
> work as we have modules which rely already on it (e.g. {{discovery}}).



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

Reply via email to