mikemccand commented on issue #12877: URL: https://github.com/apache/lucene/issues/12877#issuecomment-1856109370
> > am I making this up? > > Ha! No, you are not hallucinating @jpountz! We do have something like this for Amazon product search -- it's crucial for our usage to keep long-pole query latencies low by maximizing concurrency -- but it might just be as stupid as "setMaxMergedSegmentMB" to something "just right" for our usage. I'll poke around inside and see if our impl is not too embarrassing to share ;) OK well our (Amazon product search's) implementation is sorta messy: we subclass `TMP` and override `findMerges` to dynamically change `maxMergedSegmentMB` as a function of the total index size at this moment, and then return `super.findMerges()`. It works but it's not so clean. I would prefer for this issue that we make this a first class feature of TMP? Something like `setMinSegmentCount` or `setBalancedSegmentCount` or so? -- 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
