Repository: hive
Updated Branches:
  refs/heads/master abc1b2092 -> 92015567c


HIVE-16920: remove useless uri.getScheme() from EximUtil (Fei Hui, reviewed by 
Ferdinand Xu)


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/92015567
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/92015567
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/92015567

Branch: refs/heads/master
Commit: 92015567ceb9f31b62483faf1b0e3e19e299161d
Parents: abc1b20
Author: Ferdinand Xu <cheng.a...@intel.com>
Authored: Thu Jun 22 01:17:08 2017 +0800
Committer: Ferdinand Xu <cheng.a...@intel.com>
Committed: Thu Jun 22 01:17:08 2017 +0800

----------------------------------------------------------------------
 ql/src/java/org/apache/hadoop/hive/ql/parse/EximUtil.java | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/92015567/ql/src/java/org/apache/hadoop/hive/ql/parse/EximUtil.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/parse/EximUtil.java 
b/ql/src/java/org/apache/hadoop/hive/ql/parse/EximUtil.java
index 5b76dbd..1bff176 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/parse/EximUtil.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/parse/EximUtil.java
@@ -144,10 +144,11 @@ public class EximUtil {
     try {
       boolean testMode = conf.getBoolVar(HiveConf.ConfVars.HIVETESTMODE);
       URI uri = new Path(dcPath).toUri();
-      String scheme = uri.getScheme();
+      FileSystem fs = FileSystem.get(uri, conf);
+      // Get scheme from FileSystem
+      String scheme = fs.getScheme();
       String authority = uri.getAuthority();
       String path = uri.getPath();
-      FileSystem fs = FileSystem.get(uri, conf);
 
       LOG.info("Path before norm :" + path);
       // generate absolute path relative to home directory
@@ -161,8 +162,6 @@ public class EximUtil {
         }
       }
 
-      // Get scheme from FileSystem
-      scheme = fs.getScheme();
 
       // if scheme is specified but not authority then use the default 
authority
       if (StringUtils.isEmpty(authority)) {

Reply via email to