vigyasharma commented on issue #11676: URL: https://github.com/apache/lucene/issues/11676#issuecomment-1297303476
Sorry, I meant it'll add dependence on `nanoTime()`. I thought we use wallTime to refer to both `currentTimeInMillis` and `nanoTime`. If nanotime is acceptable, I can use something that wraps the time calls, and override them for testing, like `QueryProfilerTimer`. It will average out the time over all previous calls, instead of reporting the last call time. But maybe that's okay, given there is anyway noise in such measurements. -- 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-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
