[ 
https://issues.apache.org/jira/browse/JCRVLT-789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17934193#comment-17934193
 ] 

Julian Reschke commented on JCRVLT-789:
---------------------------------------

See 
https://issues.apache.org/jira/browse/JCRVLT-789?focusedCommentId=17933874&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-17933874

Unfortunately, that hack does work for the new test, but breaks many others. So 
we need to tune that method as well.

> AggregateImpl might be able to avoid iterating over sibling nodes
> -----------------------------------------------------------------
>
>                 Key: JCRVLT-789
>                 URL: https://issues.apache.org/jira/browse/JCRVLT-789
>             Project: Jackrabbit FileVault
>          Issue Type: Improvement
>          Components: vlt
>            Reporter: Julian Reschke
>            Assignee: Julian Reschke
>            Priority: Major
>
> See 
> [https://github.com/apache/jackrabbit-filevault/blob/367ffb423d84993c5bb0eb0186f810a58b6227be/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/AggregateImpl.java#L696]
>  
> This code currently iterates unconditionally over child nodes (which is a 
> problem for large collections). We might be able to avoid that by checking 
> the filters before descending.
> I tried a quick hack, and that made tests fail (which is good).
> Will continue with a test case first.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to