-
Notifications
You must be signed in to change notification settings - Fork 92
Open
Description
Currently the RuntimeHelper exposes two methods related to cloud
- isCompositeNodeStore and
- isCloudReadyInstance
Both namings are a bit unfortunate, because
- in cloud AEM is always started with a composite node store (even during the image build phase). The difference between image build and regular server start is which mount is the default one (apps/libs or the rest) (https://github.com/apache/jackrabbit-oak/blob/8adf6aa2d931e562ec8e69bf4261a250c53fe6f3/oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/CompositeNodeStoreService.java#L90).
- sounds a bit like distinction between AEM Classic and AEMaaCS (SDK or Cloud), but in fact what is meant here is if the composite node store is used (where only one mount is read-write)
I propose to rename
isCompositeNodeStoretoisAppsReadOnlyisCloudReadyInstancetoisCompositeNodeStore
That way it is much clearer what is meant.
Metadata
Metadata
Assignees
Labels
No labels