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