[SLING-4871] use component configurations (Config Admin) instead of framework properties

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Done
• Fix Version/s: Launchpad Builder 8, Karaf Features 1.0.0
• Component/s: Best practices, Karaf, Launchpad
• Assignee: Oliver Lietz
• Reporter: Oliver Lietz
• Created: 10/Jul/15 11:04
• Updated: 25/Jul/16 07:52
• Resolved: 07/Oct/15 14:34

Description

framework properties have several drawbacks (as discussed in SLING-4649)

• framework properties have to be set before framework start
• framework properties can not be changed at runtime
• framework properties can be shared by multiple components but are used as a last resort here only
• component configurations otherwise can be created and changed at runtime
• component configurations serve as documentation also

for components falling back to repository.home (found by rombert) additionally configurations may be required:

oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/osgi/FileBlobStoreService.java:    private static final String PROP_HOME = "repository.home";
oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/blob/datastore/AbstractDataStoreService.java:    private static final String PROP_HOME = "repository.home";
oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentNodeStoreService.java:    public static final String DIRECTORY = "repository.home";
oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndexProviderService.java:    public static final String REPOSITORY_HOME = "repository.home";

Not setting repository.home points us quickly to missing component configurations.

Issue Links

• relates to: SLING-4649 FullTextIndexingTest.testUploadedPdfIsIndexed fails on Oak