Mailing-List: contact chukwa-dev-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: chukwa-dev@hadoop.apache.org
Message-ID: <1290990637.1240693290917.JavaMail.jira@brutus>
Date: Sat, 25 Apr 2009 14:01:30 -0700 (PDT)
From: "Hudson (JIRA)" <jira@apache.org>
To: chukwa-dev@hadoop.apache.org
Subject: [jira] Commented: (CHUKWA-163) source code should not use DATACONF
 envrionment variable
In-Reply-To: <829466647.1240262212607.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/CHUKWA-163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12702785#action_12702785 ] 

Hudson commented on CHUKWA-163:
-------------------------------

Integrated in Chukwa-trunk #8 (See [http://hudson.zones.apache.org/hudson/job/Chukwa-trunk/8/])
    

> source code should not use DATACONF envrionment variable
> --------------------------------------------------------
>
>                 Key: CHUKWA-163
>                 URL: https://issues.apache.org/jira/browse/CHUKWA-163
>             Project: Hadoop Chukwa
>          Issue Type: Bug
>          Components: User Interface
>    Affects Versions: 0.1.2
>            Reporter: Terence Kwan
>            Assignee: Terence Kwan
>             Fix For: 0.1.2
>
>         Attachments: CHUKWA-163.patch
>
>
> DataConfig.java and DatabaseConfig.java should not use DATACONFIG environment variable. It should use CHUKWA_CONF_DIR.
> Index: src/java/org/apache/hadoop/chukwa/inputtools/mdl/DataConfig.java
> ===================================================================
> --- src/java/org/apache/hadoop/chukwa/inputtools/mdl/DataConfig.java	(revision 766193)
> +++ src/java/org/apache/hadoop/chukwa/inputtools/mdl/DataConfig.java	(working copy)
> @@ -26,10 +26,11 @@
>  import java.util.Iterator;
>  import java.util.HashMap;
>  import java.util.Map;
> +import java.io.File;
>  
>  public class DataConfig {
>    private static Configuration config;
> -  final static String DATACONFIG = "mdl.xml";
> +  final static String MDL_XML = "mdl.xml";
>    private Log log = LogFactory.getLog(DataConfig.class);
>  
>    public DataConfig(String path) {
> @@ -39,9 +40,11 @@
>    }
>  
>    public DataConfig() {
> -    String dataConfig = System.getenv("DATACONFIG");
> +    String dataConfig = System.getenv("CHUKWA_CONF_DIR");
>      if (dataConfig == null) {
> -      dataConfig = DATACONFIG;
> +      dataConfig = MDL_XML;
> +    } else {
> +      dataConfig += File.separator + MDL_XML;
>      }
>      log.debug("DATACONFIG=" + dataConfig);
>      if (config == null) {
> Index: src/java/org/apache/hadoop/chukwa/database/DatabaseConfig.java
> ===================================================================
> --- src/java/org/apache/hadoop/chukwa/database/DatabaseConfig.java	(revision 766193)
> +++ src/java/org/apache/hadoop/chukwa/database/DatabaseConfig.java	(working copy)
> @@ -22,6 +22,7 @@
>  import org.apache.hadoop.conf.Configuration;
>  import org.apache.hadoop.fs.Path;
>  import java.util.*;
> +import java.io.File;
>  
>  public class DatabaseConfig {
>    private Configuration config = null;
> @@ -32,6 +33,7 @@
>    public final static long MONTH = 30 * 24 * 60 * 60 * 1000L;
>    public final static long WEEK = 7 * 24 * 60 * 60 * 1000L;
>    public final static long DAY = 24 * 60 * 60 * 1000L;
> +  public final static String MDL_XML = "mdl.xml";
>  
>    public DatabaseConfig(String path) {
>      Path fileResource = new Path(path);
> @@ -40,7 +42,13 @@
>    }
>  
>    public DatabaseConfig() {
> -    Path fileResource = new Path(System.getenv("DATACONFIG"));
> +    String dataConfig = System.getenv("CHUKWA_CONF_DIR");
> +    if (dataConfig == null) {
> +      dataConfig = MDL_XML;
> +    } else {
> +      dataConfig += File.separator + MDL_XML;
> +    }
> +    Path fileResource = new Path(dataConfig);
>      config = new Configuration();
>      config.addResource(fileResource);
>    }

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