You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "Bruno Roustant (Jira)" <ji...@apache.org> on 2020/03/03 09:19:00 UTC
[jira] [Commented] (LUCENE-9257) FSTLoadMode should not be
BlockTree specific as it is used more generally in index package
[ https://issues.apache.org/jira/browse/LUCENE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17050049#comment-17050049 ]
Bruno Roustant commented on LUCENE-9257:
----------------------------------------
[~simonwillnauer] [~mikemccand] could I have your opinion on this?
> FSTLoadMode should not be BlockTree specific as it is used more generally in index package
> ------------------------------------------------------------------------------------------
>
> Key: LUCENE-9257
> URL: https://issues.apache.org/jira/browse/LUCENE-9257
> Project: Lucene - Core
> Issue Type: Improvement
> Reporter: Bruno Roustant
> Priority: Minor
> Time Spent: 10m
> Remaining Estimate: 0h
>
> 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)
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org