cnauroth commented on code in PR #5504: URL: https://github.com/apache/hive/pull/5504#discussion_r1950232995
########## standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/Warehouse.java: ########## @@ -133,14 +133,8 @@ public FileSystem getFs(Path f) throws MetaException { /** - * Hadoop File System reverse lookups paths with raw ip addresses The File - * System URI always contains the canonical DNS name of the Namenode. - * Subsequently, operations on paths with raw ip addresses cause an exception - * since they don't match the file system URI. Review Comment: I am onboard (+1 non-binding) with applying this logic to all file systems, not just blobstores. I can't think of any harm from this. -- 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-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org