[ 
https://issues.apache.org/jira/browse/HDFS-1167?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12933975#action_12933975
 ] 

Konstantin Boudnik commented on HDFS-1167:
------------------------------------------

Ok, looks like the code for this has been committed to Common a long time ago. 
The change for the config hasn't come through somehow. There should be a 
similar change for MR.

As for putting this config thing into common: as much as it might seem suitable 
we can't do it because Common's part of Herriot doesn't have any config 
processing in it ;( In fact, I think we did a wrong thing when 
{{getHadoopLocalConfDir}} has been added to Common code. This has created 
indirect dependency to the config processing logic in upstream components 
(HDFS, MR).

Therefore, I think this patch (and its counterpart) can be committed to make 
the change complete for now. However, a new JIRA needs to be created to 
refactor this method away from the Common module.

> [Herriot] New property for local conf directory in system-test-hdfs.xml file.
> -----------------------------------------------------------------------------
>
>                 Key: HDFS-1167
>                 URL: https://issues.apache.org/jira/browse/HDFS-1167
>             Project: Hadoop HDFS
>          Issue Type: Task
>          Components: test
>    Affects Versions: 0.21.0
>            Reporter: Vinay Kumar Thota
>            Assignee: Vinay Kumar Thota
>         Attachments: HDFS-1167.patch, HDFS-1167.patch
>
>
> Adding new property in system-test.xml file for local configuration directory.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to