impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sailesh Mukil (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) IMPALA-3419: DefaultFS value should be extracted from hive.metastore.warehouse.dir if fully qualified
Date Tue, 03 May 2016 20:46:48 GMT
Sailesh Mukil has uploaded a new patch set (#3).

Change subject: IMPALA-3419: DefaultFS value should be extracted from hive.metastore.warehouse.dir
if fully qualified
......................................................................

IMPALA-3419: DefaultFS value should be extracted from hive.metastore.warehouse.dir if fully
qualified

The defaultFS value in core-site.xml is ignored if the
'hive.metastore.warehouse.dir' is fully qualified in hive-site.xml.
Impala will not know about this and this could cause problems when
dealing with unqualified paths.

This patch adds a getHiveConfig() function and the backend extracts
the filesystem from the 'hive.metastore.warehouse.dir' if it is fully
qualified. Otherwise, it retrieves the 'fs.defaultFS'.

Change-Id: Id47be2b0e9c25a8bd838555aa75c949c23050c04
---
M be/src/runtime/exec-env.cc
M be/src/service/frontend.cc
M be/src/service/frontend.h
M be/src/util/hdfs-util.cc
M be/src/util/hdfs-util.h
M fe/src/main/java/com/cloudera/impala/service/JniFrontend.java
6 files changed, 61 insertions(+), 13 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/81/2881/3
-- 
To view, visit http://gerrit.cloudera.org:8080/2881
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id47be2b0e9c25a8bd838555aa75c949c23050c04
Gerrit-PatchSet: 3
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Sailesh Mukil <sailesh@cloudera.com>
Gerrit-Reviewer: Henry Robinson <henry@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sailesh@cloudera.com>

Mime
View raw message