romseygeek commented on PR #11999: URL: https://github.com/apache/lucene/pull/11999#issuecomment-1340992346
> you shouldnt search on stored fields It works surprisingly well as a confirmation phase in a two-phase query! The fact that memory indexes don't support stored fields also makes highlighting in monitor/percolator more annoying. And with Uwe's suggested refactor it's a very minimal change. It's also I think the only LeafReader functionality apart from Vectors that MemoryIndex doesn't support? It just seems a weird omission. -- 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
