-
Notifications
You must be signed in to change notification settings - Fork 4.8k
HIVE-28908: From clause supports catalog.db.tbl pattern #5774
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
|
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
|
@zhangbutao, this PR is not yet ready for review, right? |
Yes, I'll optimize this PR later. |
57fd870 to
bb7221f
Compare
|
i am not sure that could provide complete functionality without the Catalog properties support used to deduct client-impl and catalog related configs. for more info please see HIVE-29178 |
This PR is currently only an attempt to explore the scenario of federated querying across multiple internal catalogs(HIVE-26227). As for federated queries involving other catalogs such as Iceberg & JDBC catalogs (related to HIVE-29178), I believe further implementation efforts will be explored in the future. |
bb7221f to
6f8248d
Compare
13e2a71 to
46d47fb
Compare
46d47fb to
a0d39ec
Compare
a0d39ec to
55e9949
Compare
|
|
@zhangbutao Is this an abandoned PR draft? |
@aturoczy Thanks for taking care of this PR. This PR will ultimately enable capabilities related to |



Currently, this PR only focus on from clause against the HMS native catalog, and does not involve other external catalogs such as JDBC catalog or Iceberg REST catalog.
We can have some discussions based on this PR and formulate a complete solution for Hive's multi-catalog functionality
For example, we can query a iceberg table by the catalog.db.tbl pattern:
For the iceberg metadata table, we can use the catalog.db.tbl.meta pattern
select * from hive.testicedb.testice.history;What changes were proposed in this pull request?
Why are the changes needed?
Does this PR introduce any user-facing change?
How was this patch tested?