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

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

Hi Kanter,

According to your comments, I uploaded a new patch which has following updates:
1. rebase the patch on the latest trunk
2. did not edit release-log.txt
3. use the --no-prefix option in git diff
4. replace 'File.separator' codes with 'new File(path, filename)'
5. when "hadoopConfPath" is null or "", uses the default config defined by 
'oozie.service.HadoopAccessorService.hadoop.configurations'

Btw, I also agree the 3rd comment: '3. It might also be helpful to make this EL 
function available in Coordinators too'. However I suggest to implement this 
feature in another jira for coordinator only and this jira will focus on 
workflow as its title. What's your opinion?

Thanks very much!

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




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

Reply via email to