[GitHub] [lucene] dsmiley commented on pull request #205: LUCENE-9959: Rename TermVectorsReader to TermVectorsReaderBase

[GitBox <gi...@apache.org>]

dsmiley commented on pull request #205:
URL: https://github.com/apache/lucene/pull/205#issuecomment-883540136


   FWIW, what I care about most in this is that Fields.java is relegated to an internal only thing associated with the terms index, for those that build PostingsFormats 'n such.  I find it confusing to have this Fields class used in different ways across the codebase -- the PostingsFormat related Fields, and the term vector Fields.  I care less about what we name some of the things, or wether a new public class is needed somewhere.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org