[ 
https://issues.apache.org/jira/browse/SLING-3501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14054569#comment-14054569
 ] 

Georg Henzler commented on SLING-3501:
--------------------------------------

I think we can make both behaviour work, 
HealthCheckFilter.getTaggedHealthCheckServiceReferences(final String... tags) 
at the moment already receives tags that might be prefixed with "!" for tags to 
be included. We could introduce "&" and "|" as prefix and make "&" the default. 

Maybe it would be better to change the signature of 
getTaggedHealthCheckServiceReferences to not have implicit commands in the tag 
strings though, e.g. getTaggedHealthCheckServiceReferences(final String[] 
tagsToInclude, final String[] tagsToExclude, boolean combineByAnd), but this 
would mean introducing a property/checkbox "combineByAnd" to both 
org.apache.sling.hc.core.impl.CompositeHealthCheck and 
org.apache.sling.hc.webconsole.impl.HealthCheckWebconsolePlugin, so using the 
first approach is definitely easier to implement. WDYT?





> Unexpected behaviour when using multiple tags in the web console to run checks
> ------------------------------------------------------------------------------
>
>                 Key: SLING-3501
>                 URL: https://issues.apache.org/jira/browse/SLING-3501
>             Project: Sling
>          Issue Type: Bug
>          Components: Health Check
>            Reporter: Georg Henzler
>            Assignee: Bertrand Delacretaz
>         Attachments: SLING-3501-filter-for-taglist-fix.patch
>
>
> Multiple tags (e.g. "category1,category2") are connected with a logical AND 
> at the moment, that means for the given example only HCs with both category 
> tags would be run. 
> From a user's point of view this behaviour is not intuitive (IMHO, also see 
> Björn's comment in the sling-users list [1]). Using the tags 
> "category1,category2" should rather cause all checks of both categories to be 
> run (union instead of intersection). 
> The problem is evident for both the web console [2] and when configuring 
> CompositeHealthChecks (both use HealthCheckFilter).
> [1] 
> http://apache-sling.73963.n3.nabble.com/Healthcheck-Unexpected-behavior-with-tags-td4032090.html
> [2] http://localhost:4502/system/console/healthcheck



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

Reply via email to