msfroh commented on PR #15620: URL: https://github.com/apache/lucene/pull/15620#issuecomment-3887393674
> TLDR i don't handle deletes during regular merges (although I could, for us it wasn't worth the extra I/O, but for upstream I wasn't totally sure. Oh... that's a very interesting idea! For a use-case that's almost entirely append-only (like most time series), I can see how that would be the right choice. I wonder if it makes sense to add a toggle to choose this behavior? Or would that be adding too many knobs? (Even if it is configurable, I'm kind of inclined to make "ignore deletes during regular merges" the default, since that's probably what the target audience would want.) -- 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]
