[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=14043098#comment-14043098
 ] 

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

Hi Robert,

According to your comments, I updated the patch as above comment. Now user also 
could use the new function hadoop:conf() in their coordinator job. Take a 
example, user could use hadoop:conf() in coordinator.xml:
{code}
<coordinator-app name="cron-coord" frequency="${coord:minutes(10)}" 
start="${start}" end="${end}" timezone="UTC"
                 xmlns="uri:oozie:coordinator:0.2">
    <action>
        <workflow>
            <app-path>${workflowMyApp3Uri}</app-path>
            <configuration>
                <property>
                    <name>coord_map_java_opts</name>
                    <value>${hadoop:conf('jobtracker_host:9001', 
'mapreduce.map.java.opts')} -Xss539k </value>
                </property>
            </configuration>
        </workflow>
    </action>
</coordinator-app>
{code}

And then, coordinator job could also get the hadoop option.

Could you please help take a review on the new patch?

Thanks!

> 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, OOZIE-1740.patch, OOZIE-1740.patch
>
>




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