ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nikolay Izhikov (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (IGNITE-6630) Incorrect time units of average transaction commit/rollback duration cache metrics.
Date Tue, 06 Feb 2018 07:16:00 GMT

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

Nikolay Izhikov edited comment on IGNITE-6630 at 2/6/18 7:15 AM:
-----------------------------------------------------------------

[~xtern] Looks good to me.

Also, I can't open Team City build link you provide.
Can you, please, make sure the link is OK?


was (Author: nizhikov):
[~xtern] Looks good to me.

> Incorrect time units of average transaction commit/rollback duration cache metrics.
> -----------------------------------------------------------------------------------
>
>                 Key: IGNITE-6630
>                 URL: https://issues.apache.org/jira/browse/IGNITE-6630
>             Project: Ignite
>          Issue Type: Bug
>    Affects Versions: 2.2
>            Reporter: Pavel Pereslegin
>            Assignee: Pavel Pereslegin
>            Priority: Minor
>              Labels: metrics, newbie
>             Fix For: 2.5
>
>
> AverageTxCommitTime and AverageTxRollbackTime metrics in CacheMetrics counts in milliseconds
instead of microseconds as pointed in javadoc.
> Simple junit reproducer:
> {code:java}
> public class CacheMetricsTxAvgTimeTest extends GridCommonAbstractTest {
>     /** */
>     private <K, V> CacheConfiguration<K, V> cacheConfiguration(String name)
{
>         CacheConfiguration<K, V> cacheConfiguration = new CacheConfiguration<>(name);
>         cacheConfiguration.setCacheMode(CacheMode.PARTITIONED);
>         cacheConfiguration.setAtomicityMode(CacheAtomicityMode.TRANSACTIONAL);
>         cacheConfiguration.setStatisticsEnabled(true);
>         return cacheConfiguration;
>     }
>     /** */
>     public void testTxCommitDuration() throws Exception {
>         try ( Ignite node = startGrid(0)) {
>             IgniteCache<Object, Object> cache = node.createCache(cacheConfiguration(DEFAULT_CACHE_NAME));
>             try (Transaction tx = node.transactions().txStart()) {
>                 cache.put(1, 1);
>                 // Await 1 second.
>                 U.sleep(1_000);
>                 tx.commit();
>             }
>             // Documentation says that this metric is in microseconds.
>             float commitTime = cache.metrics().getAverageTxCommitTime();
>             // But this assertion will fail because it in milliseconds and returns only
~1000.
>             assert commitTime >= 1_000_000;
>         }
>     }
> }
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message