Hello Chetan,

On 15/01/2015 15:16, [email protected] wrote:
>  class IndexDefinition implements Aggregate.AggregateMapper{
> +    /**
> +     * Name of the internal property that contains the child order defined in
> +     * org.apache.jackrabbit.oak.plugins.tree.impl.TreeConstants
> +     */
> +    private static final String OAK_CHILD_ORDER = ":childOrder";
> +
>
didn't check the code actually using the constants, but wouldn't it be
possible to work directly with the Tree API if you need it to be
ordered? OAK_CHILD_ORDER is an implementation detail and should not be
used outside of core.

We could either change the Tree API by adding an isOrdered() or add a
utility method in the TreeUtil.

As I don't think we'll ever change the property `:childOrder` it's not
really needed but will clean the code.

WDYT?

Cheers
davide


Reply via email to