You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@helix.apache.org by "dafu (JIRA)" <ji...@apache.org> on 2013/02/04 03:16:14 UTC

[jira] [Updated] (HELIX-41) fix intermittent test failures

     [ https://issues.apache.org/jira/browse/HELIX-41?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

dafu updated HELIX-41:
----------------------

    Description: 
a few tests have the following pattern for assert test failure:

sleep(xx) // wait zk callback to arrive or sth. similar
assert(ret==true)
but this depends on the zk callback latency and it's not stable.

change it to:
for (int i = 0; i < 10) {
  if (ret == true)
    break;
  sleep(xx);
}
assert(ret == true)

  was:
a few tests have the following pattern for assert test failure:

sleep(xx) // wait zk callback to arrive or sth. similar
assert(ret==true)
but this depends on the zk callback latency and it's not stable.

change it to:
for (int i = 0; i < 10) {
  if (ret == true)
    break;
  sleep(xx);
}

    
> fix intermittent test failures
> ------------------------------
>
>                 Key: HELIX-41
>                 URL: https://issues.apache.org/jira/browse/HELIX-41
>             Project: Apache Helix
>          Issue Type: Bug
>            Reporter: dafu
>            Assignee: dafu
>
> a few tests have the following pattern for assert test failure:
> sleep(xx) // wait zk callback to arrive or sth. similar
> assert(ret==true)
> but this depends on the zk callback latency and it's not stable.
> change it to:
> for (int i = 0; i < 10) {
>   if (ret == true)
>     break;
>   sleep(xx);
> }
> assert(ret == true)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira