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: [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]
