[SLING-10884] i18n ResourceBundleProvider is called for every change in the repo

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: i18n 2.5.16
• Fix Version/s: i18n 2.5.18
• Component/s: i18n
• Assignee: Joerg Hoh
• Reporter: Joerg Hoh
• Created: 26/Oct/21 08:14
• Updated: 07/Nov/21 18:48
• Resolved: 01/Nov/21 16:03

Description

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.

By default /var/eventing should be excluded, so that the Sling eventing operations do not trigger this ResourceChangeListener.

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