You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Sailesh Mukil (Code Review)" <ge...@cloudera.org> on 2016/05/03 22:46:48 UTC

[Impala-CR](cdh5-trunk) IMPALA-3419: DefaultFS value should be extracted from hive.metastore.warehouse.dir if fully qualified

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 <sa...@cloudera.com>
Gerrit-Reviewer: Henry Robinson <he...@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sa...@cloudera.com>