[ https://issues.apache.org/jira/browse/SLING-2930?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Adrian Nistor updated SLING-2930: --------------------------------- Attachment: patch.diff > Wasted work in ClusterViewServiceImpl.getClusterView() > ------------------------------------------------------ > > Key: SLING-2930 > URL: https://issues.apache.org/jira/browse/SLING-2930 > Project: Sling > Issue Type: Bug > Components: Extensions > Environment: any > Reporter: Adrian Nistor > Labels: patch, performance > Attachments: patch.diff > > > The problem appears in revision 1495454. I attached a one-line patch > that fixes it. > In method "ClusterViewServiceImpl.getClusterView", the loop over > "clusterViewImpl.getInstances()" should break immediately after > "foundLocal" is set to "true". All the iterations after "foundLocal" > is set to "true" do not perform any useful work, at best they just set > "foundLocal" again to "true". > Method "isLive" in class "ResourceResolverContext" has a similar loop > (over "dynamicProviders"), and this loop breaks immediately after > "result" is set to "false", just like in the proposed patch. Other > methods (e.g., "FactoryPreconditions.checkPreconditions", > "JspUtil.checkAttributes", "EclipseJavaCompiler.compile", > "SecondPassVisitor.appendPageDirective") also have similar loops with > similar breaks, just like in the proposed patch. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira