Georg Henzler created SLING-3500:
------------------------------------

             Summary: Make CompositeHealthCheck use HealthCheckExecutor for 
parallel execution
                 Key: SLING-3500
                 URL: https://issues.apache.org/jira/browse/SLING-3500
             Project: Sling
          Issue Type: Improvement
          Components: Health Check
    Affects Versions: Health Check Core 1.1.0
            Reporter: Georg Henzler


As the CompositeHealthCheck is used fairly heavily in the well-known product 
([1] & [2]) it would be good to make the parallel execution available for the 
CompositeHealthCheck as well (this would also be in line with the web console 
that is already using the HealthCheckExecutor). 

The attached patch 
- uses the HealthCheckExecutor (shortening the implementation of the 
CompositeHealthCheck.execute() for quite a bit)
- needs to detect cycles in the configuration in a different way (the 
ThreadLocal does not work anymore)
- comes with a unit test that tests both the execution and the cycle detection

[1] 
http://localhost:4502/system/sling/monitoring/mbeans/org/apache/sling/healthcheck/HealthCheck/systemchecks.json
[2] 
http://localhost:4502/libs/granite/operations/content/hr.html/system/sling/monitoring/mbeans/org/apache/sling/healthcheck/HealthCheck/securitychecks



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to