[jira] [Commented] (SAMZA-888) SamzaContainer initializes ExponentialSleepStrategy with randInt that is inclusive of 0

["Tao Feng (JIRA)" <ji...@apache.org>]

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

Tao Feng commented on SAMZA-888:
--------------------------------

https://reviews.apache.org/r/44420/

> SamzaContainer initializes ExponentialSleepStrategy with randInt that is inclusive of 0
> ---------------------------------------------------------------------------------------
>
>                 Key: SAMZA-888
>                 URL: https://issues.apache.org/jira/browse/SAMZA-888
>             Project: Samza
>          Issue Type: Bug
>    Affects Versions: 0.10.0
>            Reporter: Navina Ramesh
>              Labels: newbie
>             Fix For: 0.10.1
>
>
> SamzaContainer initializes ExponentialSleepStrategy with randInt for initialDelay, when reading the JobModel. 
> randInt is inclusive of 0. initialDelay cannot be 0 in a backoff approach. The assert check in ExponentialSleepStrategy ends up failing the container occasionally. This should be fixed to not include 0 for initialDelay.
> As a side note: This also causes occasional build failure because the unit test for reading JobModel fails. Hence, it is important to fix this. 



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