nk1506 commented on PR #8907: URL: https://github.com/apache/iceberg/pull/8907#issuecomment-1801545652
@pvary , My Bad i didn't understand the initial commends on loading all the table data from HMS. As you mentioned initial implementation was not filtering to construct the `tableIdentifiers` for `listAllTables`. I have addressed these comments. Also followed the same strategy of `InMemoryCatalog` in terms of Error for Hive Catalog. Regarding the code duplications (`HiveTableOperations`/`HiveViewOperations`) I am creating another PR that can go before 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: 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
