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

Manjunath Ballur commented on OOZIE-1173:
-----------------------------------------

But, there is a "notNull()" method defined in "OozieClient.java" which is 
exactly similar to the "ParamChecker::notNull()" method. So, we can use this 
method, instead of "ParamChecker::notNull()".

> Refactor: use ParamChecker inXOozieClient
> -----------------------------------------
>
>                 Key: OOZIE-1173
>                 URL: https://issues.apache.org/jira/browse/OOZIE-1173
>             Project: Oozie
>          Issue Type: Task
>          Components: client
>    Affects Versions: trunk
>            Reporter: jun aoki
>            Priority: Trivial
>              Labels: newbie
>             Fix For: trunk
>
>
> In 
> [XOozieClient.java|http://svn.apache.org/viewvc/oozie/trunk/client/src/main/java/org/apache/oozie/client/XOozieClient.java?revision=1401367&view=markup],
>  you can find a piece of code like this and should be replaced with 
> ParamChecker.
> {code}
>  if (conf == null) {
>             throw new IllegalArgumentException("conf cannot be null");
> }
> {code}



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

Reply via email to