jackrabbit-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ard Schrijvers (JIRA)" <j...@apache.org>
Subject [jira] Commented: (JCR-1064) Optimize queries that check for the existence of a property
Date Fri, 24 Aug 2007 13:09:30 GMT

    [ https://issues.apache.org/jira/browse/JCR-1064?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12522508

Ard Schrijvers commented on JCR-1064:

Hmmm, having a problem. 

When the index is empty at startup, the multiIndex seems to index some documents 

if (indexNames.size() == 0) {
                reindexing = true;

This means, that 

index.getIndexReader().numDocs() == 0; is always false in SearchIndex doInit(). 

I can try to add in MultiIndex when reindexing = true to add default fieldNames.PROPERTY_SET,
but I am afraid, if somebody makes a change, it might break again. It is kind of a problem
of the 

allFieldNames.contains(FieldNames.PROPERTIES_SET) || numDocs == 0

test, which assumes some parts, and can be easy to break. 

WDOT? Shall I add it to MultiIndex, when reindexnig is true that  PROPERTY_SET FieldName is
added? I do not really like it though.

> Optimize queries that check for the existence of a property
> -----------------------------------------------------------
>                 Key: JCR-1064
>                 URL: https://issues.apache.org/jira/browse/JCR-1064
>             Project: Jackrabbit
>          Issue Type: Improvement
>          Components: indexing
>    Affects Versions: 1.3.1
>            Reporter: Ard Schrijvers
>            Priority: Minor
>             Fix For: 1.4
> //*[@mytext] is transformed into the org.apache.jackrabbit.core.query.lucene.MatchAllQuery,
that through the MatchAllWeight uses the MatchAllScorer.  The calculateDocFilter() in MatchAllScorer
 does not scale and becomes slow for growing number of nodes. 
> Solution: lucene documents will get a new Field:
> public static final String PROPERTIES_SET = "_:PROPERTIES_SET".intern();
> that holds the available properties of this document. 
> NOTE: Lucene indices build without this performance improvement should still work and
fall back to the original implementation

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message