falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Srikanth Sundarrajan (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FALCON-24) Allow re-run of timedout instances
Date Fri, 19 Jul 2013 02:32:49 GMT

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

Srikanth Sundarrajan commented on FALCON-24:
--------------------------------------------

Thanks. Have some observations on the patch.

* assertCoordStatus should attempt to check the status a few times before throwing exception
preferably with a sleep

* coord action can be in running, waiting or ready state. This needs to be checked in the
assertCoordStatus

* There is a Thread.sleep() left behind in the patch before the status is returned

* This patch may not pass Checkstyle checks. Run mvn verify before the patch is uploaded.
I have put down some guidelines relating to this. Please refer: https://cwiki.apache.org/confluence/display/FALCON/How+To+Contribute


                
> Allow re-run of timedout instances
> ----------------------------------
>
>                 Key: FALCON-24
>                 URL: https://issues.apache.org/jira/browse/FALCON-24
>             Project: Falcon
>          Issue Type: Bug
>    Affects Versions: 0.3
>            Reporter: Shwetha G S
>              Labels: rerun, timedout
>         Attachments: FALCON-24.patch
>
>
> Falcon doesn't allow re-run of timedout/failed instances of feed/process

--
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

Mime
View raw message