You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@lucene.apache.org by GitBox <gi...@apache.org> on 2021/08/16 02:55:30 UTC

[GitHub] [lucene] gf53520 commented on pull request #231: LUCENE-10041: Remove TieredMergePolicy's redundant query computation

gf53520 commented on pull request #231:
URL: https://github.com/apache/lucene/pull/231#issuecomment-899177506


   > @gf53520 Sorry for the late response. I was offline last week.
   > 
   > After taking another look, I think we should not introduce another `size` method in MergePolicy, but wrap MergeContext to cache `numDeletesToMerge` in SoftDeletesRetentionMergePolicy instead. I know this is not ideal, but it's less confusing than the current approach.
   ok, I think make that cache for SoftDeletesRetentionMergePolicy is more elegrant, considering that size method is only useful in soft-deletes case.
   


-- 
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-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org