[
https://issues.apache.org/jira/browse/IGNITE-11815?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andrew Mashenkov updated IGNITE-11815:
--------------------------------------
Description:
For now we have GridTestUtils.retryAssert() method which runs a closure 'n'
times to check if some invariantes become ok, eventually.
This method catch assertion error (this looks like a very bad idea) and can
print them to log many times even if assertion is acceptable for the moment.
Also, it is possible to miss an assertion is not related to those ones that
closure checks (e.g. assertion error thrown from ignite internals).
Let's replace retryAssert with GridTestUtils.waitForCondition() usage to make
logs clearer and to avoid possible false positive results.
was:
For now we have GridTestUtils.retryAssert() method which runs a closure 'n'
times to check if some invariantes become ok, eventually.
This method catch assertions and can print them to log many times even if
assertion is acceptable for the moment.
Also, it is possible to miss an assertion is not related to those ones that
closure checks (e.g. assertion error thrown from ignite internals).
Let's replace retryAssert with GridTestUtils.waitForCondition() usage to make
logs clearer and to avoid possible false positive results.
> Get rid of GridTestUtils.retryAssert method.
> --------------------------------------------
>
> Key: IGNITE-11815
> URL: https://issues.apache.org/jira/browse/IGNITE-11815
> Project: Ignite
> Issue Type: Test
> Reporter: Andrew Mashenkov
> Priority: Major
> Labels: MakeTeamcityGreenAgain
>
> For now we have GridTestUtils.retryAssert() method which runs a closure 'n'
> times to check if some invariantes become ok, eventually.
> This method catch assertion error (this looks like a very bad idea) and can
> print them to log many times even if assertion is acceptable for the moment.
> Also, it is possible to miss an assertion is not related to those ones that
> closure checks (e.g. assertion error thrown from ignite internals).
> Let's replace retryAssert with GridTestUtils.waitForCondition() usage to make
> logs clearer and to avoid possible false positive results.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
