Uploaded image for project: 'Jackrabbit Content Repository'
  1. Jackrabbit Content Repository
  2. JCR-4339

Indexing configuration condition property seems to be broken

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.16.0
    • Fix Version/s: 2.18, 2.17.5, 2.16.4, 2.14.7
    • Component/s: core
    • Labels:
      None

      Description

      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:

      return condition == null || condition.evaluate(state);

      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 

        Attachments

        1. JCR-4339.patch
          13 kB
          Claus Köll

          Issue Links

            Activity

              People

              • Assignee:
                c_koell Claus Köll
                Reporter:
                giacomolm Giacomo Lamonaco
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: