[ 
https://issues.apache.org/jira/browse/SOLR-6275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14076289#comment-14076289
 ] 

Mark Miller commented on SOLR-6275:
-----------------------------------

bq. hence is not suitable for time measurements.

+1, we should probably fix it. 

This is where the performance issue that I've read about with concurrent 
requests to nano time scare me though. This happens per request. If that 
happened to be a decent measurable hit, it may be better that this method 
returns bad information .0001% of the time.

> Improve accuracy of QTime reporting
> -----------------------------------
>
>                 Key: SOLR-6275
>                 URL: https://issues.apache.org/jira/browse/SOLR-6275
>             Project: Solr
>          Issue Type: Improvement
>          Components: search
>            Reporter: Ramkumar Aiyengar
>            Priority: Minor
>
> Currently, {{QTime}} uses {{currentTimeMillis}} instead of {{nanoTime}} and 
> hence is not suitable for time measurements. Further, it is really started 
> after all the dispatch logic in {{SolrDispatchFilter}} (same with the top 
> level timing reported by {{debug=timing}}) which may or may not be expensive, 
> and hence may not fully represent the time taken by the search. This is to 
> remedy both cases.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to