Tim-Brooks commented on PR #16224: URL: https://github.com/apache/lucene/pull/16224#issuecomment-4658136173
Tokenstream isn't necessarily a scenario that benefits greatly from columnar. However, the lack of support means that anyone using user provided token streams would need to fall back to the addDocument path. So it is valuable to support this scenario to prevent this required fallback. This PR kind of depends on https://github.com/apache/lucene/pull/16219 as it naturally makes sense to support multicolumn per field in case the user wants to add a stored value when the token stream is provided. -- 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]
