ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrey Gura (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (IGNITE-12028) [IEP-35] HitRateMetric should provide rateTimeInterval value to metrics exporter
Date Mon, 05 Aug 2019 11:28:00 GMT

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

Andrey Gura commented on IGNITE-12028:
--------------------------------------

[~NSAmelchev] [~NIzhikov]

Guys, could you please avoid in the future javadoc like this:

{code:java}
    /** @return Rate time interval in milliseconds. */
    public long rateTimeInterval() {
        return cntr.rateTimeInterval;
    }
{code}

It is public method so javadoc must contain method description and additionaly {{return tag}}.

> [IEP-35] HitRateMetric should provide rateTimeInterval value to metrics exporter
> --------------------------------------------------------------------------------
>
>                 Key: IGNITE-12028
>                 URL: https://issues.apache.org/jira/browse/IGNITE-12028
>             Project: Ignite
>          Issue Type: Improvement
>            Reporter: Andrey Gura
>            Assignee: Amelchev Nikita
>            Priority: Major
>              Labels: IEP-35
>             Fix For: 2.8
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> {{HitRateMetric}} allows to get only counter value while it would be useful to get also
{{rateTimeInterval}} in order to export this value as part of metric name. 
> For example look at cache metric {{RebalancingKeysRate}}. Value of this measurement could
be exported as smth like this: {{cache.<cacheName>.RebalancingKeysRate.<rateTimeInterval>
= <value>}}.
> So {{HitRateMetric}} should implement {{ObjectMetric}} interface instead of {{LongMetric}}
interface.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Mime
View raw message