[HIVE-16920] remove useless uri.getScheme() from EximUtil

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.0.0
• Fix Version/s: 3.0.0
• Component/s: Query Processor
• Target Version/s: 3.0.0
• Assignee: Hui Fei
• Reporter: Hui Fei
• Created: 20/Jun/17 09:35
• Updated: 23/May/18 00:00
• Resolved: 21/Jun/17 17:19

Description

EximUtil.java

static URI getValidatedURI(HiveConf conf, String dcPath) throws SemanticException {
    try {
      boolean testMode = conf.getBoolVar(HiveConf.ConfVars.HIVETESTMODE);
      URI uri = new Path(dcPath).toUri();
      String scheme = uri.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
      if (!path.startsWith("/")) {
        if (testMode) {
          path = (new Path(System.getProperty("test.tmp.dir"), path)).toUri().getPath();
        } else {
          path =
              (new Path(new Path("/user/" + System.getProperty("user.name")), path)).toUri()
                  .getPath();
        }
      }

      // Get scheme from FileSystem
      scheme = fs.getScheme();
  ...
}

We found that String scheme = uri.getScheme(); is useless, we can remove it.