You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Bertrand Delacretaz (JIRA)" <ji...@apache.org> on 2015/12/03 10:54:11 UTC

[jira] [Commented] (SLING-5337) Extract Startup API from Launchpad API

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

Bertrand Delacretaz commented on SLING-5337:
--------------------------------------------

We might want to discuss this design on the Sling dev list as others might have valuable input.

Splitting the various functions (readyness, startup mode) into distinct Java interfaces would help separate concerns.

> 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)