[GitHub] [lucene] romseygeek commented on pull request #11999: Add support for stored fields to MemoryIndex

[GitBox <gi...@apache.org>]

romseygeek commented on PR #11999:
URL: https://github.com/apache/lucene/pull/11999#issuecomment-1340643868

   MemoryIndex is used by the monitor module and by the ES percolator as well, and that's where we may have issues - there are some slow queries in ES that use stored fields to search non-indexed documents, and they won't work when run against a MemoryIndex.
   
   We need to store the types of the data so that we can call the correct methods on StoredFieldsVisitor.  But you're right, it might be easier to store as a Map and then use instanceof checks to get the types - will give that a go and see if it works!


-- 
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-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org