[LUCENE-3552] TaxonomyReader/Writer and their Lucene* implementation

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 3.5, 4.0-ALPHA
• Component/s: modules/facet
• Lucene Fields: New, Patch Available
• Assignee: Shai Erera
• Reporter: Shai Erera
• Created: 02/Nov/11 07:17
• Updated: 28/Aug/22 13:00
• Resolved: 03/Nov/11 06:29

Description

The facet module contains two interfaces TaxonomyWriter and TaxonomyReader, with two implementations Lucene*. We've never actually implemented two TaxonomyWriters/Readers, so I'm not sure if these interfaces are useful anymore. Therefore I'd like to propose that we do either of the following:

1. Remove the interfaces and remove the Lucene part from the implementation classes (to end up with TW/TR impls). Or,
2. Keep the interfaces, but rename the Lucene* impls to Directory*.

Whatever we do, I'd like to make the impls/interfaces impl also TwoPhaseCommit.

Any preferences?