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

Yi Pan (Data Infrastructure) commented on SAMZA-893:
----------------------------------------------------

[~jmaes], thanks for splitting the patches. I would recommend to open a 
separate JIRA for the unit test fix, s.t. we can commit and close on this one.

+1 on the logic fix already.

> Fix a bug with host affinity request expiration introduced in SAMZA-867
> -----------------------------------------------------------------------
>
>                 Key: SAMZA-893
>                 URL: https://issues.apache.org/jira/browse/SAMZA-893
>             Project: Samza
>          Issue Type: Bug
>            Reporter: Jake Maes
>            Assignee: Jake Maes
>         Attachments: SAMZA-893.patch, SAMZA-893_logic.patch
>
>
> The expiration logic change in the patch for SAMZA-867 was wrong. It 
> simplified the conditional, but would cause expired requests to never get 
> rescheduled on ANY_HOST. 
> This ticket is to fix that logic and the unit tests, which didn't fail after 
> the change. 



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

Reply via email to