[OAK-7510] Run repository initializers with hooks

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.10.0, 1.9.9
• Component/s: core
• Labels: modularization
• Assignee: Alex Deparvu
• Reporter: Alex Deparvu
• Created: 25/May/18 09:10
• Updated: 08/Oct/19 15:21
• Resolved: 07/Sep/18 14:40

Description

Currently the repository initializers (RepositoryInitializer and WorkspaceInitializer) run when the repo boots without any hooks [0] which means that current RepositoryInitializers need to setup custom Roots (roots with hardcoded editor providers like NamespaceEditorProvider and TypeEditorProvider) on top of the provided builders to be able to setup properly. I'm looking at the InitialContent [1] and the CugConfiguration [2] in the context of installing node types.

I would like to look into removing the hardcoded providers and trying to run all existing editors over the content produced by the initializers.

As an added benefit this will allow decoupling of hard dependencies between components (see for example OAK-7499)

[0] https://github.com/apache/jackrabbit-oak/blob/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/Oak.java#L687
[1] https://github.com/apache/jackrabbit-oak/blob/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/InitialContent.java#L134
[2] https://github.com/apache/jackrabbit-oak/blob/trunk/oak-authorization-cug/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/cug/impl/CugConfiguration.java#L162

Issue Links

• is blocked by: OAK-7527 Drop mandatory constraint on "rep:system" node for "rep:privileges"
• is related to: OAK-7779 Ignore copying of namespace index node in InitialContentMigrator
• is related to: OAK-7786 Make the NamespaceEditor less strict when enforcing changes to rep:nsdata node
• relates to: OAK-7499 Dependencies on various 'plugins'

Sub-Tasks

• Move InitialContent#INITIAL_CONTENT to test: Closed
• Define and register oak namespace: Closed
• VersionStorageEditor should allow bootstraping of initial content: Closed
• NameValidator is not using namespaces defined in the current transaction: Closed
• Allow for initial registration of 'xml' namespace: Closed