[
https://issues.apache.org/jira/browse/IGNITE-6630?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16204039#comment-16204039
]
ASF GitHub Bot commented on IGNITE-6630:
----------------------------------------
GitHub user xtern opened a pull request:
https://github.com/apache/ignite/pull/2854
IGNITE-6630 Time units fix.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/xtern/ignite IGNITE-6630
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/ignite/pull/2854.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2854
----
commit 73cdd7c96914b3a08551f918e2214a5fdb7a4390
Author: Pereslegin Pavel <[email protected]>
Date: 2017-10-13T19:01:40Z
IGNITE-6630 Time units fix.
----
> 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
> Reporter: Pavel Pereslegin
> Assignee: Pavel Pereslegin
> Priority: Minor
> Labels: metrics, newbie
>
> AverageTxCommitTime and AverageTxRollbackTime metrics in CacheMetrics
> calculated 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
(v6.4.14#64029)
