Details
-
Improvement
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
None
-
None
-
None
-
New
Description
FSTLoadMode and its associate attribute key (static String) are currently defined in BlockTreeTermsReader, but they are actually used outside of BlockTree in the general "index" package.
CheckIndex and ReadersAndUpdates are using these enum and attribute key to drive the FST load mode through the SegmentReader which is not specific to a postings format. They have an unnecessary dependency to BlockTreeTermsReader.
We could move FSTLoadMode out of BlockTreeTermsReader, to make it a public enum of the "index" package. That way CheckIndex and ReadersAndUpdates do not import anymore BlockTreeTermsReader.
This would also allow other postings formats to use the same enum (e.g. LUCENE-9254)