rmuir commented on issue #12203:
URL: https://github.com/apache/lucene/issues/12203#issuecomment-1508457296

   Check it out here for string fields to see what i mean:
   
https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/codecs/DocValuesConsumer.java#L662-L663
   
https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/codecs/DocValuesConsumer.java#L849-L850
   
   it would be bad to instantiate a ton of these OrdinalMap instances (one for 
each field) all at once, rather than sequentially which keeps the memory low.


-- 
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

Reply via email to