You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Joerg Hoh (Jira)" <ji...@apache.org> on 2021/10/26 08:15:00 UTC

[jira] [Created] (SLING-10884) i18n ResourceBundleProvider is called for every change in the repo

Joerg Hoh created SLING-10884:
---------------------------------

             Summary: i18n ResourceBundleProvider is called for every change in the repo
                 Key: SLING-10884
                 URL: https://issues.apache.org/jira/browse/SLING-10884
             Project: Sling
          Issue Type: Improvement
          Components: i18n
    Affects Versions: i18n 2.5.16
            Reporter: Joerg Hoh


The i18n ResourceBundleProvider is registered as ResourceChangeListener and configured to listen to all changes within the repository.
And if the change does not affect a languageRoot (that's probably the vast majority of changes), it will always open (and close) a ResourceResolver. This comes with some overhead (500-1000 µs, depending on setup).

It should be possible to exclude some paths from being considered at all by configuration (maybe already at the level of the registration of the properties of the ResourceChangeListener, otherwise as part of the {{onChange}} method), so that the creation of this ResourceResolver can be avoided. In this case no ResourceBundles will be detected below these configured paths.
The default should remain as is.

(Although I think it could be beneficial to add /var/eventing and /var/jobs by default to these paths, as there shouldn't be ResourceBundles there, and also in these paths there are a lot of changes if Sling Jobs are used.)


[1] https://github.com/apache/sling-org-apache-sling-i18n/blob/ca5472b409c2b0485af6f8a97db521ad6694e927/src/main/java/org/apache/sling/i18n/impl/JcrResourceBundleProvider.java#L265



--
This message was sent by Atlassian Jira
(v8.3.4#803005)