rmuir commented on PR #11999: URL: https://github.com/apache/lucene/pull/11999#issuecomment-1341303231
> The elasticsearch highlighter for the percolator has been working for years with a really annoying workaround, because MemoryIndex is the only LeafReader implementation in lucene that doesn't support the `document` method, and that this small PR will allow me to remove. Thanks @romseygeek this is an actual valid use-case (as opposed to searchiing on stored fields which is NOT). please proceed. Though I will mention, its not true about only LeafReader without `document`, trust me, there are others, i know (look in IndexingChain.java if you need an example). -- 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
