original-brownbear opened a new pull request, #13885: URL: https://github.com/apache/lucene/pull/13885
Lazy initialize these fields. They consume/cause a lot of memory/GC because they are allocated frequently (~7% of all allocations in luceneutil's wikimedia medium run for me). This does not cause any measurable slowdown as far as runtime is concerned and since these are not even needed for all instances (in fact they are rarely used in the queries the benchmark explores) save quite a few CPU cycles for collecting and allocating them. @jpountz I know we talked about reusing these instances, but as far as I can tell this deals with the problem much more fundamentally, we simply often do not need these? -- 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
