You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@oozie.apache.org by "sam liu (JIRA)" <ji...@apache.org> on 2014/05/05 09:52:15 UTC

[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

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