msfroh commented on issue #14887: URL: https://github.com/apache/lucene/issues/14887#issuecomment-3029546887
Yeah... that makes sense. The overhead of adding an integer field to filtering/wrapping `DocIdSetIterator`s is the downside that jumps out to me, but that's (I think) at most a 50% increase in the size of a very small object (assuming it has a 32-bit reference to its own class and a 32-bit reference to the wrapped/filtered iterator). I guess there's also copying the docId into the field just the read it out. Intuitively, though, I can see how the benefit of monomorphic inlining could/would outweigh that. 👍 -- 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
