omkenge commented on PR #1251: URL: https://github.com/apache/iceberg-python/pull/1251#issuecomment-2442663561
Hi @kevinjqliu , ### Previous Situation In the existing implementation of the list_tables method within hive catalog, the functionality aimed to retrieve all tables under a specified namespace. However, it did not account for the distinction between Iceberg tables and views. Specifically, the method included both Iceberg tables and virtual views in its results, which could lead to confusion and incorrect data handling in downstream processes. ### Changes Made To improve the accuracy of the list_tables method, the following changes have been implemented: **1) Exclusion of Views: The condition for filtering tables now explicitly excludes views by checking that the tableType is not "VIRTUAL_VIEW"** **2) Implemented list_views Method: A new list_views method has been implemented to specifically retrieve Iceberg views under the given namespace. This method ensures that only valid Iceberg views are returned** -- 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]
