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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
