[GitHub] [lucene] mikemccand commented on pull request #11796: GITHUB#11795: Add FilterDirectory to track write amplification factor

[GitBox <gi...@apache.org>]

mikemccand commented on PR #11796:
URL: https://github.com/apache/lucene/pull/11796#issuecomment-1255217103

   I love this approach/idea!
   
   It's simple so we should start with this ... but it will necessarily be a lagging indicator since merging takes some time to kick off and run to completely while flushing keeps happening if docs are being indexed.  Also, it reports the "for all time" WAF instead of adding some decay and being closer to an instantaneous measure.  But we can try to improve those later.
   
   Have you tried turning this on for a `luceneutil` indexing run to see what WAF it reports?  It might be tricky to do it right because by default I think `luceneutil` indexing does not wait for merges while indexing.


-- 
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