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

Robert Kanter commented on OOZIE-1600:
--------------------------------------

Can you refactor it slightly so that there's a single return statement at the 
end instead of one in the if statement?   Otherwise looks good, though there 
seems to be something wrong with Jenkins...

> map-reduce actions without configuration section in workflow.xml throws 
> "IllegalArgumentException: element cannot be null"
> --------------------------------------------------------------------------------------------------------------------------
>
>                 Key: OOZIE-1600
>                 URL: https://issues.apache.org/jira/browse/OOZIE-1600
>             Project: Oozie
>          Issue Type: Bug
>            Reporter: Bowen Zhang
>            Assignee: Bowen Zhang
>         Attachments: oozie-1600.patch, oozie-1600.patch
>
>
> In evaluateConfigurationProperty() method of MapReduceActionExecutor.java, we 
> cannot assume "configuration" is always there. If the element is null, we 
> need to return defaultValue.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to