You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by "Julian Reschke (JIRA)" <ji...@apache.org> on 2019/05/07 09:09:00 UTC

[jira] [Updated] (JCR-4339) Indexing configuration condition property seems to be broken

     [ https://issues.apache.org/jira/browse/JCR-4339?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Julian Reschke updated JCR-4339:
--------------------------------
    Labels:   (was: candidate_jcr_2_14)

> Indexing configuration condition property seems to be broken 
> -------------------------------------------------------------
>
>                 Key: JCR-4339
>                 URL: https://issues.apache.org/jira/browse/JCR-4339
>             Project: Jackrabbit Content Repository
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 2.16.0
>            Reporter: Giacomo Lamonaco
>            Assignee: Claus Köll
>            Priority: Major
>             Fix For: 2.18, 2.17.5, 2.16.4, 2.14.7
>
>         Attachments: JCR-4339.patch
>
>
> Hi, while upgrading jackrabbit library from 2.14 to 2.16 we noticed couple of issues with the indexing configuration [1]. More specifically one of this issue is related to the condition property. 
> While using the latest library the value of the condition property is totally ignored, this means that the index-rule is applied to all nodes with the specified nodetype.
> We did some investigations and maybe we found the root cause in this commit [2]. More specifically, if you have a look at line 805 of the IndexingConfigurationImpl, the appliesTo method is always returning true: it looks not correct to me. 
> As a quick workaround we replace that line with:
> {code:java}
> return condition == null || condition.evaluate(state);{code}
> After this small fix the condition property in the index rules is working again as expected. Please let us if this is a bug or a misunderstanding from our side. 
> [1][https://wiki.apache.org/jackrabbit/IndexingConfiguration]
> [2][https://github.com/apache/jackrabbit/commit/a7804ede88623ceaa3b62f638f5384ebb97b39d4#diff-a79478132fc8ad1fe2bce5fa64f45310R805] 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)