kevinjqliu commented on issue #2052: URL: https://github.com/apache/iceberg-python/issues/2052#issuecomment-2931706304
Thanks for reporting this! For context here is where the `.db` suffix is appended. https://github.com/apache/iceberg-python/blob/91853898c89f5376aa7fe874e8c2c6eb2ca224b2/pyiceberg/catalog/__init__.py#L931-L933 I believe this is done to mirror the hive/HMS convention. https://github.com/apache/hive/blob/de994e5c556930d1761601a61d6941c69fa9e153/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/Warehouse.java#L74 I dont think we need/want to do this as part of the base catalog class. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
