[LUCENE-4565] Simplify TaxoReader ParentArray/ChildrenArrays

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 4.1, 6.0
• Component/s: modules/facet
• Lucene Fields: New, Patch Available
• Assignee: Shai Erera
• Reporter: Shai Erera
• Created: 19/Nov/12 14:44
• Updated: 28/Aug/22 13:32
• Resolved: 06/Dec/12 14:06

Description

TaxoReader exposes two structures which provide information about a categories parent/childs/siblings: ParentArray and ChildrenArrays. ChildrenArrays are derived (i.e. created) from ParentArray.

I propose to consolidate all that into one API ParentInfo, or CategoryTreeInfo (a better name?) which will provide the same information, only from one object. So instead of making these calls:

int[] parents = taxoReader.getParentArray();
int[] youngestChilds = taxoReader.getChildrenArrays().getYoungestChildArray();
int[] olderSiblings = taxoReader.getChildrenArrays().getOlderSiblingArray();

one would make these calls:

int[] parents = taxoReader.getParentInfo().parents();
int[] youngestChilds = taxoReader.getParentInfo().youngestChilds();
int[] olderSiblings = taxoReader.getParentInfo().olderSiblings();

Not a big change, just consolidate more code into one logical place. All of these arrays will continue to be lazily allocated.