[
https://issues.apache.org/jira/browse/SLING-11601?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610942#comment-17610942
]
Joerg Hoh edited comment on SLING-11601 at 10/27/22 4:00 PM:
-------------------------------------------------------------
PR for Scripting Sightly:
https://github.com/apache/sling-org-apache-sling-scripting-sightly/pull/21
(merged)
(I was not able to convert all instances to LazyBindings, because the
LazyBindings do not offer a matching Constructor to just accept the un-wrapped
values).
was (Author: joerghoh):
PR for Scripting Sightly:
https://github.com/apache/sling-org-apache-sling-scripting-sightly/pull/21
(I was not able to convert all instances to LazyBindings, because the
LazyBindings do not offer a matching Constructor to just accept the un-wrapped
values).
> [Scripting] Use the LazyBindings instead of SimpleBindings
> ----------------------------------------------------------
>
> Key: SLING-11601
> URL: https://issues.apache.org/jira/browse/SLING-11601
> Project: Sling
> Issue Type: Improvement
> Components: Sling Models
> Affects Versions: Sling Models Impl 1.4.0
> Reporter: Joerg Hoh
> Assignee: Joerg Hoh
> Priority: Major
> Time Spent: 1h 50m
> Remaining Estimate: 0h
>
> At various modules Sling Scripting (and Sling Models) code still uses a
> {{SimpleBindings}} implementation of the {{{}Bindings{}}}, while it could use
> {{LazyBindings}} which are more efficient in case not all provided bindings
> are required.
> While the scripting itself does not necessarily use a
> {{LazyBindings.Supplier}}, other {{BindingValueProviders}} can be invoked on
> them, which do support the LazyBindings; and in that case there can be a
> benefit.
> Requires a bump of the dependency to Sling API though (at least 2.21.0 is
> required)
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
