[Iceberg]Prefix table (loaded from Hive Catalog) with catalog name #25140
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR add the catalog name to the name of Iceberg table that loaded from Hive catalog. Currently in Presto, Iceberg tables loaded from Native catalog (Hadoop, Nessie, Rest) have the table name of format
<catalog>.<schema>.<table>
, however, Iceberg tables loaded from Hive catalog have the table name of format<schema>.<table>
. This may lead in some inconsistencies across different catalogs. For example, the test case in PR #24904 relies on the name of Iceberg table loaded from different catalogs.Moreover, Iceberg lib itself has a consistent behavior across all catalogs, that is return the full table name including catalog name. See here.
Motivation and Context
Ensure that the names of tables that loaded from different catalogs have consistent formatting
Impact
The names of Iceberg tables loaded from Hive catalog always include catalog names.
Test Plan
TestIcebergDistributedHive
to verify that the table name loaded from Hive catalog contains catalog nameContributor checklist
Release Notes