Joerg Hoh created SLING-10269:
---------------------------------

             Summary: improve effiency of isResourceType()
                 Key: SLING-10269
                 URL: https://issues.apache.org/jira/browse/SLING-10269
             Project: Sling
          Issue Type: Improvement
          Components: ResourceResolver
    Affects Versions: Resource Resolver 1.7.2
            Reporter: Joerg Hoh
            Assignee: Joerg Hoh


I have code, which uses {noformat}resourceResolver.isResourceType(Resource, 
String){noformat} very often to determine type information. I also observed, 
that the execution time of this method contributes a lot to the overall 
execution time.

Also the number of unique resourcetypes which are going to be checked 
(resource.getResourceType, 1st parameter) is typically quite low; the same 
applies for the number of resourcetypes it is compared against. So it makes 
sense to cache the result of that method call in map which shares the lifetime 
of the ResourceResolver, as the result will never change during this lifetime. 
But during a refresh() this map can be cleared, so any changes in the RT 
hierarchy which might have happened will get effective.

I have tested this approach already and it gives a significant speedup to my 
code (reduced execution time by 50%); of course this cannot be expected 
universally, as this is an extreme case, but it shows that there is indeed a 
bit of overhead.





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

Reply via email to