hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Konstantin Boudnik (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HDFS-1167) [Herriot] New property for local conf directory in system-test-hdfs.xml file.
Date Fri, 19 Nov 2010 20:47:13 GMT

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


Mime
View raw message