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