Pulkitg64 commented on PR #15003: URL: https://github.com/apache/lucene/pull/15003#issuecomment-3504205131
> OK, I think your plan from this comment (50% check) is the way to go for now. My intuition is that the relative threshold is likely better, but we would need to make the percentage lower (like 25%, or 20%, like your other benchmarks show). Since in my approach, even if graph has no deletes, it will still try to reconnect nodes which will make merging slower, hence I agree with you that "relative threshold" is likely better. Sharing benchmark results in some time > As for "Reconstruct completely", yes, lets focus on 40% or fewer deleted total docs? Does that align with your benchmarking results? Keeping this threshold at 40% now. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
