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

Jan Høydahl commented on SOLR-16073:
------------------------------------

You propose to drop 1m, 5m, 15m only in the solr-exporter output and not in 
/admin/metrics, right?

> totalTime metric should be milliseconds (not nano)
> --------------------------------------------------
>
>                 Key: SOLR-16073
>                 URL: https://issues.apache.org/jira/browse/SOLR-16073
>             Project: Solr
>          Issue Type: Bug
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: metrics
>            Reporter: David Smiley
>            Priority: Minor
>
> I observed that the "totalTime" metric has been a nanosecond number in recent 
> years, yet once upon a time it was milliseconds. This change was very likely 
> inadvertent. Our prometheus solr-exporter-config.xml shows that it thinks 
> it's milliseconds. It's not; RequestHandlerBase increments this counter by 
> "elapsed", the response of timer.stop() -- nanoseconds. Years ago it had 
> invoked {{MetricUtils.nsToMs}} but it appears [~ab] removed this as a part of 
> other changes in 2017 sometime -- 
> https://github.com/apache/solr/commit/d8df9f8c9963c2fc1718fd471316bf5d964125ba
> Also, I question the value/purpose of this metric.  Is it so useful that it 
> deserves to be among our relatively few metrics exported in our default 
> prometheus exporter config?  It's been there since the initial config but I 
> wonder why anyone wants it.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to