You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by "Konrad Windszus (Jira)" <ji...@apache.org> on 2022/09/12 07:42:00 UTC

[jira] [Updated] (JCRVLT-524) JcrPackageRegistry.register should always register the subpackages as well

     [ https://issues.apache.org/jira/browse/JCRVLT-524?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Konrad Windszus updated JCRVLT-524:
-----------------------------------
    Fix Version/s: 3.6.4
                       (was: 3.6.2)

> JcrPackageRegistry.register should always register the subpackages as well
> --------------------------------------------------------------------------
>
>                 Key: JCRVLT-524
>                 URL: https://issues.apache.org/jira/browse/JCRVLT-524
>             Project: Jackrabbit FileVault
>          Issue Type: Improvement
>          Components: vlt
>            Reporter: Konrad Windszus
>            Priority: Major
>             Fix For: 3.6.4
>
>
> The {{FSPackageRegistry}} already registers all contained subpackages with a call to {{PackageRegistry.register(...)}} (https://github.com/apache/jackrabbit-filevault/blob/14615ce5647252005f2271b5f5f0351eb91aa78e/vault-core/src/main/java/org/apache/jackrabbit/vault/packaging/registry/impl/FSPackageRegistry.java#L362) both for internal and external files, while {{JcrPackageRegistry.register(...)}} does not do that (https://github.com/apache/jackrabbit-filevault/blob/14615ce5647252005f2271b5f5f0351eb91aa78e/vault-core/src/main/java/org/apache/jackrabbit/vault/packaging/registry/impl/JcrPackageRegistry.java#L348).
> The handling should be consolidated so that all implementations always register also the contained subpackages and the javadoc of {{PackageRegistry.register(...)}} clarified accordingly.
> Registering subpackages probably requires extracting them from the container package and store as dedicated package which leads to increased memory consumption.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)