[OAK-4806] Remove usage of Tree in LuceneIndexEditor

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.5.11, 1.6.0
• Component/s: lucene
• Labels: performance
• Assignee: Chetan Mehrotra
• Reporter: Chetan Mehrotra
• Created: 15/Sep/16 04:19
• Updated: 04/Oct/16 08:48
• Resolved: 16/Sep/16 07:02

Description

LuceneIndexEditor currently creates 2 tree instances for determining IndexRule. ianeboston highlighted this on list [1] and this is something which we should avoid and remove usage of Tree api

This was earlier done so as to simplify future support for conditional rules (OAK-2281) which might need access to ancestor which is not possible with NodeState api. As that is not going to be done so we can get rid of Tree construction in the editor.

[1] https://lists.apache.org/thread.html/7d51b45296f5801c3b510a30a4847ce297707fb4e0d4c2cefe19be62@%3Coak-dev.jackrabbit.apache.org%3E

Issue Links

• relates to: OAK-2281 Support for conditional indexingRules