[
https://issues.apache.org/jira/browse/SLING-8408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16837265#comment-16837265
]
Thomas Mueller commented on SLING-8408:
---------------------------------------
The exception can also be thrown if the query syntax is wrong, and actually
that's possible and should be resolved with SLING-8413. But I think it's fine
to fail the health check in this case as well.
I will create a pull request.
> DistributionQueueHealthCheck should deal with failing queries
> -------------------------------------------------------------
>
> Key: SLING-8408
> URL: https://issues.apache.org/jira/browse/SLING-8408
> Project: Sling
> Issue Type: Improvement
> Components: Content Distribution
> Reporter: Thomas Mueller
> Priority: Major
>
> The following health check indirectly runs a queries which might fail:
> *
> [DistributionQueueHealthCheck|https://github.com/apache/sling-org-apache-sling-distribution-core/blob/master/src/main/java/org/apache/sling/distribution/monitor/DistributionQueueHealthCheck.java]:
>
> sling-org-apache-sling-distribution-core/src/main/java/org/apache/sling/distribution/monitor
> The call
> [JobManagerImpl.findJobs|https://github.com/apache/sling-org-apache-sling-event/blob/master/src/main/java/org/apache/sling/event/impl/jobs/JobManagerImpl.java#L373],
> which can throw an exception with SLING-8407, if the index is not yet
> available. The health checks should catch this exception and return
> HEALTH_CHECK_ERROR for this case.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
