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

Sreepathi Prasanna commented on OOZIE-1945:
-------------------------------------------

I think the changes are simple and does not require a testcase.

> NPE in JaveActionExecutor#check()
> ---------------------------------
>
>                 Key: OOZIE-1945
>                 URL: https://issues.apache.org/jira/browse/OOZIE-1945
>             Project: Oozie
>          Issue Type: Bug
>    Affects Versions: trunk
>            Reporter: Mona Chitnis
>            Priority: Trivial
>             Fix For: trunk, 4.1.0
>
>         Attachments: OOZIE-1945-patch.3, oozie-1945-patch-2, 
> oozie-1945.patch.1
>
>
> in method check()
> {code}
>  String errorCode = props.getProperty("error.code");
>                             if (errorCode.equals("0")) {
>                                 errorCode = "JA018";
>                             }
>                             if (errorCode.equals("-1")) {
>                                 errorCode = "JA019";
>                             }
>                             errorReason = props.getProperty("error.reason");
> {code}
> if error.code is null, these leads to NPEs
> easy fix
> {code}
> if ("0".equals(errorCode))
> ...
> {code}



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

Reply via email to