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

Robert Kanter commented on OOZIE-1653:
--------------------------------------

Mostly looks good to me.  Three minor things:
# The test method, {{testRetryAllErroCode}}, has a typo in the name and should 
be {{testRetryAllErrorCode}}
# Can you update the oozie-site/default descriptions for 
{{oozie.service.LiteWorkflowStoreService.user.retry.error.code.ext}} and 
{{oozie.service.LiteWorkflowStoreService.user.retry.error.code}} to mention the 
"ALL" special value?
# For thoroughness, the test should also cover the case where "ALL" is 
specified in {{oozie.service.LiteWorkflowStoreService.user.retry.error.code}}

> Support ALL to allowed error code of the user retry
> ---------------------------------------------------
>
>                 Key: OOZIE-1653
>                 URL: https://issues.apache.org/jira/browse/OOZIE-1653
>             Project: Oozie
>          Issue Type: New Feature
>          Components: action
>    Affects Versions: 4.0.0
>            Reporter: Azrael
>            Priority: Minor
>         Attachments: OOZIE-1653.1.patch, OOZIE-1653.2.patch, OOZIE-1653.patch
>
>
> Action retry for all the errors if ALL is configured.
> oozie-site.xml
> {code:xml}
> <name>oozie.service.LiteWorkflowStoreService.user.retry.error.code.ext</name>
> <value>ALL</value>
> {code}
> This feature make convenient to set error handle policy.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to