[jira] [Commented] (IGNITE-4007) SQL: QueryMetrics.minimumTime() is always zero.

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on IGNITE-4007:
----------------------------------------

GitHub user akuznetsov-gridgain opened a pull request:

    https://github.com/apache/ignite/pull/1134

    IGNITE-4007 Fixed Query metrics min time

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/gridgain/apache-ignite ignite-4007

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/ignite/pull/1134.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 #1134
    
----
commit b9105df77cc70b532f94899c754fba47e3e05e9a
Author: vozerov-gridgain <vo...@gridgain.com>
Date:   2016-09-28T11:53:20Z

    IGNITE-3989: Moved failing JTA tests to ignore module.

commit d595345765db2151ff432beecd478ce056393593
Author: vozerov-gridgain <vo...@gridgain.com>
Date:   2016-09-28T12:08:38Z

    IGNITE-3990: Moved failing Spring tests to "ignore" module.

commit e3f13455d4273e615727d0410783e3719db98f76
Author: sboikov <sb...@gridgain.com>
Date:   2016-09-28T09:56:17Z

    ignite-2833 Need call 'touch' for cache entry if it was obtained using 'entryEx'.
    
    (cherry picked from commit 17c2fc0)

commit b2faa339acb2eea24e6dd5e0c21fc3d3d0592ff6
Author: sboikov <sb...@gridgain.com>
Date:   2016-09-28T10:47:51Z

    ignite-3621 Fixed 'testEvictExpired'.
    
    (cherry picked from commit 4ff19c2)

commit 74d2fc2416b8e6bc0598152552021f984a013061
Author: sboikov <sb...@gridgain.com>
Date:   2016-09-28T11:31:24Z

    ignite-3621 Fixed 'testEvictExpired'.
    
    (cherry picked from commit bfe4458)

commit d2563dacceea61b19bb6e083e29ebacc28fdd323
Author: vozerov-gridgain <vo...@gridgain.com>
Date:   2016-09-28T12:51:55Z

    IGNITE-3993: Added failing client tests to "ignored" test suite.

commit 33d34941390c00e8d6a2488e8f2e11e6abba8a01
Author: sboikov <sb...@gridgain.com>
Date:   2016-09-28T12:54:52Z

    Merge remote-tracking branch 'community/ignite-1.6.9' into ignite-1.6.9

commit 78144c4c9d6200ceef8b666a186039685f053381
Author: vozerov-gridgain <vo...@gridgain.com>
Date:   2016-09-28T13:52:13Z

    Fixed incorrect test count calculation leading to afterTestsStopped() not being called.

commit e3dfdecc3607b5f3183bfcb1ce36c57543a8965f
Author: Alexander Paschenko <al...@gmail.com>
Date:   2016-09-28T13:46:46Z

    Validate hash code presence in BinaryObject. Fixes #928

commit d1e3a78ae569fa5d5692816db44f2c677e1b8283
Author: Valentin Kulichenko <va...@gmail.com>
Date:   2016-09-29T00:53:42Z

    IGNITE-3191 - Fixed ordering of fields in binary objects

commit b280c3efa1eb84c6bc8abbe31ba669b0c24323d8
Author: vozerov-gridgain <vo...@gridgain.com>
Date:   2016-09-29T07:23:54Z

    Fixed missing Apache header.

commit 3e8a1c6b045c231dbb3c972463000f824386aee9
Author: vozerov-gridgain <vo...@gridgain.com>
Date:   2016-09-29T08:08:23Z

    Added node stop to IgniteCacheExpireAndUpdateConsistencyTest.

commit 062b9b69aa851d5253dfb7f44066b7d749f1ca12
Author: Alexey Goncharuk <al...@gmail.com>
Date:   2016-09-29T08:26:50Z

    IGNITE-3633 - Enforce key validation for tests.

commit ad613afd7165ed2d8bf0a62805fc2506dfe6c59a
Author: sboikov <sb...@gridgain.com>
Date:   2016-09-29T09:04:44Z

    ignite-3621 Fixed 'testEvictExpired'.
    
    (cherry picked from commit a4d7aa3)

commit 22dc2c9e05e5183f838865023c9cb7a8291ac67f
Author: Alexander Paschenko <al...@gmail.com>
Date:   2016-09-29T12:43:20Z

    GridNearCacheAdapter, GridDhtCacheAdapter - test fix

commit f447559ecba55149452e4e48e2bf44ef1fa1b4d7
Author: Alexey Goncharuk <al...@gmail.com>
Date:   2016-09-29T14:10:14Z

    ignite-1.6.9 - Fixing tests

commit 0957640a89f02c0a3bc834b0bc436741fdb80c1c
Author: AKuznetsov <ak...@gridgain.com>
Date:   2016-09-29T16:53:34Z

    IGNITE-4007 Fixed update of QueryMetrics.minimumTime() metric. Tests added.

----


> SQL: QueryMetrics.minimumTime() is always zero.
> -----------------------------------------------
>
>                 Key: IGNITE-4007
>                 URL: https://issues.apache.org/jira/browse/IGNITE-4007
>             Project: Ignite
>          Issue Type: Bug
>          Components: SQL
>    Affects Versions: 1.6
>            Reporter: Alexey Kuznetsov
>            Assignee: Semen Boikov
>             Fix For: 1.8
>
>
>  QueryMetrics.minimumTime() is always zero, because internal AtomicInteger initialized with zero and code that change minTime looks like
> {code}minTime.setIfLess(duration);{code}
> Obviously that execution time is always >=0 and any number is always is greate than zero and metrics never updated.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)