[jira] [Commented] (OOZIE-1740) Add a new function hadoop:conf() that can be invoked from the workflow.xml and will return a hadoop configuration option

[sam liu (JIRA)]

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

sam liu commented on OOZIE-1740:
--------------------------------

Hi Robert,

Thanks and I will update the patch according to your comments.

For 2nd feedback, does the following updated code match your comments? 

        public static String hadoop_conf(String hadoopConfPath, String 
propName) {
                Configuration conf = new Configuration();
                if (hadoopConfPath == null || hadoopConfPath.equals(""))
                        hadoopConfPath = 
Services.get().getConf().get("oozie.service.HadoopAccessorService.hadoop.configurations");

                loadHadoopConf(hadoopConfPath, conf);

                return conf.get(propName);
        }

Please let me know if I have any misunderstanding.

Thanks a lot!

> Add a new function hadoop:conf() that can be invoked from the workflow.xml 
> and will return a hadoop configuration option
> ------------------------------------------------------------------------------------------------------------------------
>
>                 Key: OOZIE-1740
>                 URL: https://issues.apache.org/jira/browse/OOZIE-1740
>             Project: Oozie
>          Issue Type: Improvement
>          Components: workflow
>    Affects Versions: trunk, 3.3.2
>            Reporter: sam liu
>            Assignee: sam liu
>         Attachments: OOZIE-1740.patch
>
>




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