Tim-Brooks opened a new pull request, #16269: URL: https://github.com/apache/lucene/pull/16269
FreqProxTermsWriter#addField now builds the downstream TermVectorsConsumerPerField only when the field stores term vectors, instead of for every indexed field. The generic `nextPerField` link in TermsHashPerField is renamed to `termVectorsPerField` and may now be null. The "term-vector sub-options without term vectors" check moves from TermVectorsConsumerPerField#start to the schema layer (IndexingChain#verifyNoTermVectorOptionsWithoutVectors), and the now-dead doVectors branching in #start is collapsed. TermsHash#startDocument/finishDocument only delegated to the term-vectors consumer, so they are removed and IndexingChain calls termVectorsWriter directly. There is no behavioral change. This is a clean-up to make the cross-field dependencies clear in order to support text inversion in the column pass. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
