luyuncheng commented on PR #11987: URL: https://github.com/apache/lucene/pull/11987#issuecomment-1332336907
> are you able to report the retrieval time as well? runStoredFieldsBenchmark.py | retrieved_time_msec | Baseline | Candidate | | :--- | :----: | ---: | | BEST_SPEED | 260.71 | 283.29 | | BEST_COMPRESSION | 2586.35 | 2553.10 | > if we create too much pressure on GC for unoptimized merges. I will do this benchmark. > we may want to explicitly supply smaller heap We have been met a situation: a es node with max_shards_per_node default 1000, and normally one shard with 40 segments, one buffer with retained heap: 100KB, so this would use 4G resident heap memory. i think with smaller heap, this PR would cause more frequent GC when segment number is not big to influent the resident heap. -- 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
