talatuyarer commented on code in PR #12768: URL: https://github.com/apache/iceberg/pull/12768#discussion_r2042544968
########## core/src/test/java/org/apache/iceberg/catalog/CatalogTests.java: ########## @@ -1224,6 +1229,9 @@ public void testListTables() { @Test public void listNamespacesWithEmptyNamespace() { + assumeThat(supportsEmptyNamespace()) Review Comment: I am working on BigQuery Catalog. In the context of the BigQuery catalog, empty namespaces are not permitted as namespaces are a required component. I believe Snowflake has a similar requirement. I want to use CatalogTests for BigQuery Catalog. I believe we should check if Catalog support empty namespace support or not which I suspect is not supported by all catalog implementations. -- 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