stefanvodita commented on issue #12989: URL: https://github.com/apache/lucene/issues/12989#issuecomment-1879610708
Thanks @msfroh! The PR looks neat and you might be right that, while `IntBlockPool` basically maintains an `int[][]` like our `ChunkedArray`, it is a bit inconveient to work with. I left more detailed comments on the PR, but the high-level question is if we've actually reduced the memory footprint during taxonomy refreshes. It's very possible I'm missing something, but right now it looks to me like we haven't improved on that front. Doing shallow copies of the old array without allocating new memory would solve it though. -- 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
