[GitHub] [hive] ayushtkn commented on a diff in pull request #4651: HIVE-27664: AlterTableSetLocationAnalyzer threw a confusing exception…

["ayushtkn (via GitHub)" <gi...@apache.org>]

ayushtkn commented on code in PR #4651:
URL: https://github.com/apache/hive/pull/4651#discussion_r1319478956


##########
ql/src/java/org/apache/hadoop/hive/ql/ddl/table/storage/set/location/AlterTableSetLocationAnalyzer.java:
##########
@@ -56,8 +56,7 @@ protected void analyzeCommand(TableName tableName, Map<String, String> partition
     } catch (FileNotFoundException e) {
       // Only check host/port pair is valid, whether the file exist or not does not matter
     } catch (Exception e) {
-      throw new SemanticException("Cannot connect to namenode, please check if host/port pair for " + newLocation +
-          " is valid", e);
+      throw new SemanticException("Unable to get path:" + newLocation , e);

Review Comment:
   Unable to get Path isn't clear either, FNF is ignored, that means path is of least importance, but the connection to FileSystem layer is.
   ```
   Can not connect to the server, please check if host/port pair for  + newLocation +
             " is valid", e);```
   
   might be a possibility as Namenode is an HDFS only concept



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org