lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gilad Barkai (JIRA)" <>
Subject [jira] [Updated] (LUCENE-5316) Taxonomy tree traversing improvement
Date Wed, 13 Nov 2013 12:03:23 GMT


Gilad Barkai updated LUCENE-5316:

    Attachment: LUCENE-5316.patch

Attaching a fixed patch with the concurrency resolved. 
This is not yet the patch with the {{int[]}} instead of an array.

I'm not sure that following the path of the int[] is right, it will block all future extensions
for using e.g on-disk children.. 
What do you think? Perhaps it is better to keep the iterator?

> Taxonomy tree traversing improvement
> ------------------------------------
>                 Key: LUCENE-5316
>                 URL:
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: modules/facet
>            Reporter: Gilad Barkai
>            Priority: Minor
>         Attachments: LUCENE-5316.patch, LUCENE-5316.patch, LUCENE-5316.patch, LUCENE-5316.patch
> The taxonomy traversing is done today utilizing the {{ParallelTaxonomyArrays}}. In particular,
two taxonomy-size {{int}} arrays which hold for each ordinal it's (array #1) youngest child
and (array #2) older sibling.
> This is a compact way of holding the tree information in memory, but it's not perfect:
> * Large (8 bytes per ordinal in memory)
> * Exposes internal implementation
> * Utilizing these arrays for tree traversing is not straight forward
> * Lose reference locality while traversing (the array is accessed in increasing only
entries, but they may be distant from one another)
> * In NRT, a reopen is always (not worst case) done at O(Taxonomy-size)
> This issue is about making the traversing more easy, the code more readable, and open
it for future improvements (i.e memory footprint and NRT cost) - without changing any of the
> A later issue(s?) could be opened to address the gaps once this one is done.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message