Re: [PR] Adding profiling support for concurrent segment search [lucene]

["jainankitk (via GitHub)" <gi...@apache.org>]

jainankitk commented on PR #14413:
URL: https://github.com/apache/lucene/pull/14413#issuecomment-3098034331

   > Sorry, I had lost track of this PR. I think that my only concern left is how it computes end times by adding the approximate timing to the start time. I'd rather not report it since this profiler doesn't actually compute an end time, but I'd be fine with reporting the sum of approximate timings if we think that it helps.
   
   No worries @jpountz, thanks for the review. Even I was away for a bit, will try to address the comments and push new revision this week.
   
   


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