You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Konrad Windszus (JIRA)" <ji...@apache.org> on 2017/03/16 13:07:41 UTC

[jira] [Commented] (SLING-6653) Make slingLaunchpadOak() private

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

Konrad Windszus commented on SLING-6653:
----------------------------------------

[~olli] WDYT?

> Make slingLaunchpadOak() private
> --------------------------------
>
>                 Key: SLING-6653
>                 URL: https://issues.apache.org/jira/browse/SLING-6653
>             Project: Sling
>          Issue Type: Improvement
>          Components: Testing
>    Affects Versions: Testing PaxExam 0.0.2
>            Reporter: Konrad Windszus
>
> The method {{slingLaunchpadOak()}} should never be used directly instead there should always be a concrete backend being set up (i.e. {{slingLaunchpadOakTar(...)}} or {{slingLaunchpadOakMongo(...)}} should be used). I would propose to make this method private and to always require the path to the working directory as argument. Then the {{LuceneIndexProviderService}} could be configured correctly i.e. as absolute path in slingLaunchpadOak(...) instead of doing it twice (once incorrect with a relative path in {{slingLaunchpadOak(...)}} and once correct with the absolute path in either {{slingLaunchpadOakMongo(...)}} and {{slingLaunchpadOakTar()}})



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)