jpountz merged PR #14189:
URL: https://github.com/apache/lucene/pull/14189
--
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.apa
jainankitk commented on PR #14189:
URL: https://github.com/apache/lucene/pull/14189#issuecomment-2686186026
Any reason this PR is not merged and got marked as stale?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
jpountz commented on PR #14189:
URL: https://github.com/apache/lucene/pull/14189#issuecomment-2686190876
None other than me being a bit anxious about side-effects, e.g. this floor
segment size also affects the behavior of merge-on-full-flush. But it shouldn't
be a big deal. I'll merge short
jainankitk commented on PR #14189:
URL: https://github.com/apache/lucene/pull/14189#issuecomment-2686184062
> For reference, this is roughly a 10x increase of the floor segment size,
so given that `TieredMergePolicy` defaults to 10 segments per tier, indexes
should have about 10 fewer segme
github-actions[bot] commented on PR #14189:
URL: https://github.com/apache/lucene/pull/14189#issuecomment-2664271800
This PR has not had activity in the past 2 weeks, labeling it as stale. If
the PR is waiting for review, notify the d...@lucene.apache.org list. Thank you
for your contributi
jpountz commented on PR #14189:
URL: https://github.com/apache/lucene/pull/14189#issuecomment-2631518128
For reference, this is roughly a 10x increase of the floor segment size, so
given that `TieredMergePolicy` defaults to 10 segments per tier, indexes should
have about 10 fewer segments a
jpountz commented on PR #14189:
URL: https://github.com/apache/lucene/pull/14189#issuecomment-2631514489
Thanks for the feedback, I was hesitating. Let's pull this in 10.2 then.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
