You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by ad...@apache.org on 2019/08/21 17:59:32 UTC

[kudu] 01/02: [mini_hms] remove unused duplicate from env_vars

This is an automated email from the ASF dual-hosted git repository.

adar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git

commit 0c9204c41cad73e64ef5427c67034f0a6a2265e9
Author: Alexey Serbin <al...@apache.org>
AuthorDate: Tue Aug 20 14:45:00 2019 -0700

    [mini_hms] remove unused duplicate from env_vars
    
    Once the code was changed in 76b80ec01, the duplicate assignment
    for the "HADOOP_CONF_DIR" environment variable was not effectively
    put into the 'env_vars' map.
    
    This patch does not contain any functional modifications.
    
    Change-Id: I3a7fb897aac5671a6938b73c66aac2d5d93fe8f7
    Reviewed-on: http://gerrit.cloudera.org:8080/14109
    Reviewed-by: Grant Henke <gr...@apache.org>
    Tested-by: Kudu Jenkins
---
 src/kudu/hms/mini_hms.cc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/kudu/hms/mini_hms.cc b/src/kudu/hms/mini_hms.cc
index 98db795..1174229 100644
--- a/src/kudu/hms/mini_hms.cc
+++ b/src/kudu/hms/mini_hms.cc
@@ -153,7 +153,6 @@ Status MiniHms::Start() {
       { "HIVE_AUX_JARS_PATH", aux_jars },
       { "HIVE_CONF_DIR", data_root_ },
       { "JAVA_TOOL_OPTIONS", java_options },
-      { "HADOOP_CONF_DIR", data_root_ },
       // Set HADOOP_OS_TYPE=Linux due to HADOOP-8719.
       // TODO(ghenke): Remove after HADOOP-15966 is available (Hadoop 3.1.3+)
       { "HADOOP_OS_TYPE", "Linux" }