Skip to content

Conversation

zhangbutao
Copy link
Contributor

What changes were proposed in this pull request?

TODO: Add more test and cover more api.

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Copy link

sonarqubecloud bot commented Oct 2, 2025

@Neer393
Copy link
Contributor

Neer393 commented Oct 3, 2025

@zhangbutao I think we need a default database created whenever a user creates new catalog so that ideally when a user switches catalog, we point to the default database within it
What do you think ?

@zhangbutao
Copy link
Contributor Author

zhangbutao commented Oct 3, 2025

@zhangbutao I think we need a default database created whenever a user creates new catalog so that ideally when a user switches catalog, we point to the default database within it What do you think ?

@Neer393 Currently(without this PR), when a new catalog is created, a default database will also be created.

With this PR, when a user switches catalogs using SET CATALOG catName, it should not point to the default database by default. The user should explicitly specify a database through USE dbName or USE catName.dbName xxx to maintain maximum compatibility with previous usage patterns.
Please check catalog_database.q https://github.com/apache/hive/pull/6088/files#diff-0a99f71cb21eb68f9efe36ff59f2005f696ef9ea6b660aef5cc184152018f79b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants