You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Georg Henzler (JIRA)" <ji...@apache.org> on 2014/07/08 08:33:36 UTC

[jira] [Commented] (SLING-3501) Unexpected behaviour when using multiple tags in the web console to run checks

    [ 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)