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

Robert Kanter commented on OOZIE-2012:
--------------------------------------

I think it would be better to have 
{{CoordCommandUtils#getNextValidActionTimeForCronFrequency}} throw the 
exception instead of returning {{null}}.  

Also, can you add a unit test?

> coordinator with an invalid cron frequency throws NPE after validation
> ----------------------------------------------------------------------
>
>                 Key: OOZIE-2012
>                 URL: https://issues.apache.org/jira/browse/OOZIE-2012
>             Project: Oozie
>          Issue Type: Bug
>            Reporter: Bowen Zhang
>            Assignee: Bowen Zhang
>             Fix For: trunk
>
>         Attachments: oozie-2012.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to