Hi, >1. Read NodeState children in orderable manner - This is required for reading indexing rules where the order is important
Add an index property on the definition? you can sort after reading all of them, there won't be too many so performance is not a concern. > Access to ancestors to evaluate condition in index rule - JR2 index rule support conditional rule where condition requires access to parent Can you post an example? How many levels down do you expect to have in a definition? I would just convert the definition to an tree-like structure made out of definitions, not content trees. best, alex On Fri, Nov 14, 2014 at 10:07 AM, Chetan Mehrotra <chetan.mehro...@gmail.com > wrote: > I need to create Tree out of NodeState for following reason > > 1. Read NodeState children in orderable manner - This is required for > reading indexing rules where the order is important > 2. Access to ancestors to evaluate condition in index rule - JR2 index > rule support conditional rule where condition requires access to > parent > > > Instead I suggest to export a generic mechanism for obtaining > (immutable) Tree instances given a node state. > Any details on how that should be done. I just need to convert a > NodeState to a Tree also support access to parent tree. > > > Chetan Mehrotra > > > On Fri, Nov 14, 2014 at 2:14 PM, Michael Dürig <mdue...@apache.org> wrote: > > > > -1. > > > > ImmutableTree is an internal class and as Felix points out exposes a lot > of > > idiosyncrasies. Further exposure will make it very difficult to evolve. > > Instead I suggest to export a generic mechanism for obtaining (immutable) > > Tree instances given a node state. This will in the long run allow us to > > make ImmutableTree package-private (as MutableTree) and only rely on that > > generic mechanism. > > > > Michael > > > > > > On 14.11.14 7:53 , Chetan Mehrotra wrote: > >> > >> Hi Team, > >> > >> For OAK-2261 I need to use ImmutableTree from > >> org.apache.jackrabbit.oak.plugins.tree package which is currently not > >> exported by Oak core. To proceed further I would like to export this > >> package as part of OAK-2269 > >> > >> Unless someone objects I would make the change by EOD today > >> > >> Chetan Mehrotra > >> > > >
