jpountz commented on PR #14604: URL: https://github.com/apache/lucene/pull/14604#issuecomment-2866464348
Can you add a CHANGES entry for this? Similar changes have not always been well received in the past, because it increases the cost of opening a reader, even though the reader may not need the field that is pre-loaded. I personally think that we should do this, because Lucene is a search library and we want to move costs to open-time rather than run-time whenever possible, but this is a notable change in runtime behavior, so I'd like to make sure it's visible in the release notes. -- 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-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
